Skip to content

AAI-268 fix token verification: need to allow for different issuers#46

Merged
marius-mather merged 7 commits into
mainfrom
token-issuer-fix
Aug 6, 2025
Merged

AAI-268 fix token verification: need to allow for different issuers#46
marius-mather merged 7 commits into
mainfrom
token-issuer-fix

Conversation

@marius-mather

@marius-mather marius-mather commented Aug 6, 2025

Copy link
Copy Markdown
Collaborator

Description

AAI-268: while trying to use the backend API to sync Auth0 roles, discovered that the backend doesn't handle the token issuer well - tokens from Auth0 can either have the tenant domain (tenant.au.auth0.com) or the custom domain (mydomain.org) as their issuer. We probably want to allow for both (may want to get tokens from Auth0 in scripts, or log in via AAI Portal), so add an extra config option to allow this.

Changes

  • Add auth0_issuer setting
  • Check both the auth0 domain and the optional issuer setting when verifying tokens
  • Unit tests of token verification

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have added unit / integration tests that prove my fix is effective or that my feature works
  • I have run all tests locally and they pass
  • I have updated the documentation (if applicable)

How to Test Manually (if necessary)

Run uv run pytest

This comment was marked as off-topic.

Comment thread .github/workflows/build-ecr.yml
@marius-mather marius-mather merged commit 0ef4a91 into main Aug 6, 2025
2 checks passed
@amandazhuyilan amandazhuyilan deleted the token-issuer-fix branch June 19, 2026 03:22
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.

3 participants