Skip to content

Conversation

@panta-123
Copy link
Contributor

Update token doc for server (single vo and multi vo) and client .
According to changes from parent issue : rucio/rucio#7310

Copy link
Contributor

@rdimaio rdimaio left a comment

Choose a reason for hiding this comment

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

Thanks for addressing my comments; LGTM on my side, but I'll leave the approval to someone with more context on OIDC

@voetberg
Copy link
Contributor

@mlassnig ?

@mlassnig
Copy link
Contributor

@mlassnig ?

LGTM, I leave signoff to @dchristidis

@bari12 bari12 requested a review from dchristidis March 25, 2025 14:14
@voetberg
Copy link
Contributor

voetberg commented May 8, 2025

ping on this @dchristidis ?

@voetberg
Copy link
Contributor

It's been a few months, I think we can merge this. @rdimaio ?

@panta-123
Copy link
Contributor Author

The main PR : rucio/rucio#7310 never got reviewed and thus missed the release.
This PR is dependent on that, so have to wait.

Copy link

@mgajek-cern mgajek-cern left a comment

Choose a reason for hiding this comment

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

Good job. Just have a few questions

Comment on lines +154 to +156
auth_type = oidc
oidc_polling = True
oidc_issuer=\<issuer's_nickname>
Copy link

@mgajek-cern mgajek-cern Oct 10, 2025

Choose a reason for hiding this comment

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

Also wanted to check: Can all configurable settings be overridden using environment variables? If so, it might make sense to either update this documentation page or create a separate document listing all the environment variables just to keep things clear and well-organized.

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.

5 participants