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

XrdSciTokens mapping name not forwarded to chained authz #1569

Closed
esindril opened this issue Dec 8, 2021 · 2 comments
Closed

XrdSciTokens mapping name not forwarded to chained authz #1569

esindril opened this issue Dec 8, 2021 · 2 comments

Comments

@esindril
Copy link
Contributor

esindril commented Dec 8, 2021

When the mapping is successful but the scope authz failed the chained authz plugin is called without the mapping info. I am referring to the following line:

https://github.com/xrootd/xrootd/blob/master/src/XrdSciTokens/XrdSciTokensAccess.cc#L408

How is the chained authz plugin expected to extract the mapping information from a token that it might not be able to decode? It would help if the new username that is now populating the Entity->eaAPI[request.name] should also be set for the new_secentity that is passed to the OnMissing call.

@bbockelm
Copy link
Contributor

bbockelm commented Dec 9, 2021

Wouldn't this largely negate the whole point of having restrictions in the token?

That is, if we populate the username and you got access to your home directory -- despite having a scopes that do not allow access to the home directory -- it would be a very surprising outcome!

@abh3 abh3 added the Discussion label Jan 4, 2022
@abh3
Copy link
Member

abh3 commented Sep 30, 2022

I think this discussion has essentially stalled. There has been a lot of evolvement of the token handling as well as definition. So, likely this is no longer relevant.Please feel free to start a new discussion.

@abh3 abh3 closed this as completed Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants