Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User auth plugins #321

Merged
merged 4 commits into from May 11, 2023
Merged

Conversation

SergeyYakubov
Copy link

Allows to configure plugins for user authentication/authorization. Includes methods to authenticate based on OIDC token (see issue galaxyproject/galaxy#15526).

This is a stacked PR after #320

@mvdbeek
Copy link
Member

mvdbeek commented May 2, 2023

This looks good to me, besides the conflict that needs resolving I think my only comment is that it would be nice to have a small example of a configuration in the docs and how to verify it works.

as job launch parameter so that it is sent to Pulsar from Galaxy. See corresponding PR (galaxyproject/galaxy#15300) and issue (galaxyproject/galaxy#15526) for Galaxy
Includes methods to authenticate based on OIDC token (see issue galaxyproject/galaxy#15526)
@mvdbeek
Copy link
Member

mvdbeek commented May 4, 2023

I think you need to add pulsar.user_auth to https://github.com/galaxyproject/pulsar/blob/master/setup.py#L57 (or use setuptools.find_packages instead of the old manual way).

@mvdbeek mvdbeek merged commit 31d8a99 into galaxyproject:master May 11, 2023
12 of 14 checks passed
@mvdbeek mvdbeek mentioned this pull request May 11, 2023
@mvdbeek
Copy link
Member

mvdbeek commented May 11, 2023

Thanks so much @SergeyYakubov, I'm sure we'll get the corresponding galaxy part in before the next release

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.

None yet

2 participants