Skip to content

Resolves Issue #1901, #1902, #1903: Fix registry auth repository fallback paths#1904

Open
afoote-mitre wants to merge 5 commits into
devfrom
af-1901
Open

Resolves Issue #1901, #1902, #1903: Fix registry auth repository fallback paths#1904
afoote-mitre wants to merge 5 commits into
devfrom
af-1901

Conversation

@afoote-mitre

@afoote-mitre afoote-mitre commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Closes Issue #1901, #1902, #1903

Summary

Fixes three repository/auth-context issues that prevented some registry and legacy compatibility paths from working correctly, and updates unit test fixtures to reflect requester UUID lookup behavior.

Important Changes

src/repositories/userRepository.js

  • Fixed getUserUUID() to pass options as the fourth argument to utils.getUserUUID().

src/utils/authContext.js

  • Updated method detection so ES6 class prototype methods are recognized correctly.

src/repositories/baseOrgRepository.js

  • Fixed legacy selected short-name lookup to instantiate OrgRepository before calling instance methods.

test/unit-tests/*

  • Added and updated unit coverage for user UUID forwarding, auth context prototype method detection, legacy org lookup, and CVE rejection requester lookup.

Testing

Steps to manually test updated functionality, if possible

  • 1. Run bash -i -c "npm run test:unit-test".
  • 2. Verify registry org GET tests pass for regular users and org admins requesting expand=users.
  • 3. Verify CVE rejection tests pass for both Secretariat and regular requester paths.

@afoote-mitre afoote-mitre requested a review from david-rocca July 2, 2026 15:26
@afoote-mitre afoote-mitre self-assigned this Jul 2, 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