Skip to content

Conversation

marius-mather
Copy link
Contributor

cc @nuwang

I previously added an option to override the entire authentication pipeline (which is useful for testing). However, for preserving the default pipeline but adding some extra processing, it may be better (and easier to maintain) to just configure additional steps that happen after the default pipeline.

I've added a config option oidc_auth_pipeline_extra to the galaxy.yml schema to support this.

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-actions github-actions bot added area/testing area/auth Authentication and authorization labels Sep 2, 2025
@github-actions github-actions bot added this to the 25.1 milestone Sep 2, 2025
Copy link
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.

This looks good to me - we are more likely to need to add stuff after decoding the access token.

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 kind/enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants