Skip to content

Resolves issue #1673, Handling read-after-writes in the registry user controller#1823

Open
afoote-mitre wants to merge 2 commits into
devfrom
af-1673
Open

Resolves issue #1673, Handling read-after-writes in the registry user controller#1823
afoote-mitre wants to merge 2 commits into
devfrom
af-1673

Conversation

@afoote-mitre
Copy link
Copy Markdown
Collaborator

Closes Issue #1673

Summary

Properly handling read-after-writes in the registryUserController, through the use of casualConsistency and some minor reordering of calls

Important Changes

src/controller/registry-user.controller/registry-user.controller.js

  • Added casualConsistency: false to updateUser(), grantRole(), and revokeRole()
  • Moved the generation of updatedUserUUID before the mutation execution blocks, matching the process for requestingUserUUID

Testing

Steps to manually test updated functionality, if possible

  • 1) Run npm run test:integration and ensure all tests pass
  • 2) Run npm run test:unit-tests and ensure all tests pass

@afoote-mitre afoote-mitre requested a review from david-rocca May 22, 2026 16:06
@afoote-mitre afoote-mitre self-assigned this May 22, 2026
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.

1 participant