Skip to content

Conversation

OlegoO
Copy link
Contributor

@OlegoO OlegoO commented Jul 8, 2025

Description

fix: Prevents auditable fields from modifications. This ensures more accurate tracking of audit fields.
fix: Prevents PasswordExpired, LockoutEnabled, LockoutEnd, LastLoginDate, and LastPasswordChangedDate from modification via User API.
fix: Update ReturnPasswordHash setting in appsettings.json. Changed the ReturnPasswordHash value in the Authorization section from true to false. Set it to true if you use legacy vc-storefront.

References

QA-test:

Jira-link:

https://virtocommerce.atlassian.net/browse/VCST-3430

Artifact URL:

Image tag:
3.900.0-pr-2923-f399-vcst-3430-f3992e69

fix: Prevents auditable fields from modifications. This ensures more accurate tracking of audit fields.
@OlegoO OlegoO requested a review from artem-dudarev July 8, 2025 14:03
Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.401
Timestamp: 08-07-2025T14:14:15

…ate, and LastPasswordChangedDate from modification via User API.
Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.518
Timestamp: 08-07-2025T16:44:40

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.933
Timestamp: 09-07-2025T13:05:27

…he `ReturnPasswordHash` value in the `Authorization` section from `true` to `false`.
Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 8.278
Timestamp: 09-07-2025T14:22:18

Copy link

Please retry analysis of this Pull-Request directly on SonarQube Cloud

Copy link
Contributor

@vc-ci vc-ci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test Suite: Test Suites/Modules/module_Assets
Tests: 13
Failures: 0
Errors: 0
Time: 7.745
Timestamp: 21-07-2025T14:56:12

@OlegoO OlegoO merged commit 108ea46 into dev Jul 24, 2025
5 of 6 checks passed
@OlegoO OlegoO deleted the fix/VCST-3430 branch July 24, 2025 06:53
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants