Skip to content

UID2-XXXX: accept euid.admin.ss-portal Okta scope#645

Merged
BehnamMozafari merged 1 commit into
mainfrom
bmz/kcc-aid-euid-okta-scope
May 21, 2026
Merged

UID2-XXXX: accept euid.admin.ss-portal Okta scope#645
BehnamMozafari merged 1 commit into
mainfrom
bmz/kcc-aid-euid-okta-scope

Conversation

@BehnamMozafari
Copy link
Copy Markdown
Contributor

Summary

Adds enum entry EUID_SS_PORTAL to OktaCustomScope so the uid2-admin auth middleware accepts the new euid.admin.ss-portal scope. This is the M2M scope the EUID self-serve portal will present when calling the admin service.

Part of the EUID Portal MVP (AI Innovation day) — Task 2 of the plan. No Jira ticket exists yet; using UID2-XXXX as a placeholder per the user's hackathon convention.

Same Role.SHARING_PORTAL mapping as the existing SS_PORTAL/uid2.admin.ss-portal entry — no new role, just a second scope name accepted.

Spec: https://thetradedesk.atlassian.net/wiki/spaces/UID2/pages/1724874898

Test plan

  • Unit test added: OktaCustomScopeTest.euidSsPortalScopeIsRecognised (verifies fromName("euid.admin.ss-portal") returns the new enum with SHARING_PORTAL role)
  • mvn test -Dtest=OktaCustomScopeTest,AdminAuthMiddlewareTest — all 31 tests pass (5 OktaCustomScopeTest + 26 AdminAuthMiddlewareTest)
  • Full mvn test — only pre-existing flakes failed in parallel-run (SiteSyncJobTest timing race, two "Bind Address already in use" port collisions). All three pass when re-run in isolation. None touch auth code.
  • Reviewer to confirm no other consumers of OktaCustomScope need updating

Adds EUID_SS_PORTAL enum entry to OktaCustomScope so the uid2-admin
auth middleware accepts the new euid.admin.ss-portal scope. Same
Role.SHARING_PORTAL mapping as the existing uid2.admin.ss-portal entry.
@BehnamMozafari BehnamMozafari merged commit bc14c0e into main May 21, 2026
4 checks passed
@BehnamMozafari BehnamMozafari deleted the bmz/kcc-aid-euid-okta-scope branch May 21, 2026 04:21
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.

2 participants