Skip to content

Better Auth0 OIDC support#22426

Open
marius-mather wants to merge 4 commits intogalaxyproject:devfrom
AustralianBioCommons:oidc-auth0-support
Open

Better Auth0 OIDC support#22426
marius-mather wants to merge 4 commits intogalaxyproject:devfrom
AustralianBioCommons:oidc-auth0-support

Conversation

@marius-mather
Copy link
Copy Markdown
Contributor

Adding a specific provider/subclass for Auth0 OIDC (I also added an Auth0 OIDC class to python-social-auth). In our Auth0 implementation we have so far relied on the generic OIDC provider, but it's helpful to have a specific provider class for it, particularly to get refresh tokens working correctly.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@github-project-automation github-project-automation bot moved this to Needs Review in Galaxy Dev - weeklies Apr 9, 2026
@github-actions github-actions bot added area/testing area/auth Authentication and authorization labels Apr 9, 2026
@github-actions github-actions bot added this to the 26.1 milestone Apr 9, 2026
Copy link
Copy Markdown
Member

@nuwang nuwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marius-mather I think it's be great to make the default GalaxyOpenIDConnect more generic if we can, rather than introduce more providers. Assuming Auth0 is largely compliant with generic oidc, what are we missing in the base provider, and can we add it so other providers can also make use of it?

@marius-mather
Copy link
Copy Markdown
Contributor Author

@nuwang one of the main things I did when adding an Auth0 OIDC backend to python-social-auth was allow it to be configured via the Auth0 domain - which is a lot more convenient, but not 100% necessary. There's also some mapping of the Auth0 profile fields so we can get full profile details.

In terms of the galaxy implementation and making it more generic, I guess the main thing that would be needed would be the option to set social-auth's EXTRA_DATA fields, which could be clunky in the XML as it can involve nested lists.

I think having an Auth0 specific provider class is cleaner, and relies less on the somewhat clunky XML parsing currently used for backend config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/auth Authentication and authorization area/testing

Projects

Status: Needs Review

Development

Successfully merging this pull request may close these issues.

2 participants