-
Notifications
You must be signed in to change notification settings - Fork 854
VCST-3430: Prevent auditable fields from modifications. #2923
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
fix: Prevents auditable fields from modifications. This ensures more accurate tracking of audit fields.
There was a problem hiding this 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.
There was a problem hiding this 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
There was a problem hiding this 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`.
There was a problem hiding this 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
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
There was a problem hiding this 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
|
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