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

Add support for BEARER_TOKEN_FILE. #13

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

bbockelm
Copy link
Contributor

The WLCG bearer token discovery protocol includes the BEARER_TOKEN_FILE environment variable which indicates a filename the token should be loaded from. This allows tokens to be updated throughout the runtime of the gfal2 library invocations.

The WLCG bearer token discovery protoccol includes the BEARER_TOKEN_FILE
environment variable which indicates a filename the token should be
loaded from.  This allows tokens to be updated throughout the runtime
of the gfal2 library invocations.
@bbockelm
Copy link
Contributor Author

@mpatrascoiu - any thoughts on this one?

@mpatrascoiu
Copy link
Contributor

Hello Brian,

Sorry, I didn't get the time to review this.
I'll try and have a look next week.

This implements support for the BEARER_TOKEN_FILE environment variable.
Does it also implement the full bearer token discovery? (specification)

@bbockelm
Copy link
Contributor Author

bbockelm commented Dec 2, 2022

@mpatrascoiu - yes, with this PR all the parts of the bearer token discovery file should be implemented.

@mpatrascoiu
Copy link
Contributor

I'll assign this ticket to @Shubhangi-Misra. There is other work in parallel, but it's good to have this assigned.

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