-
Notifications
You must be signed in to change notification settings - Fork 237
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
Support public read / auth write #3481
Support public read / auth write #3481
Conversation
…public read/auth write (SOFTWARE-5727)
…ated for a namespace with PUBLIC authorization)
This is currently deployed on topology-itb. |
@brianhlin , @djw8605 , can you review this or suggest other reviewers? |
@mwestphall can you take a look at this sometime over the next week or so? |
I'd like to have this in before I start work on https://opensciencegrid.atlassian.net/browse/SOFTWARE-5760 since they affect the same code. |
Do you think I can help somehow? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Can I start to test it? |
This is deployed in production now; go ahead. |
Looks ok, still testing. |
SOFTWARE-5727; even if a namespace has PUBLIC auth, if it also has SciTokens auth information, generate a scitokens.conf for the origin but make sure "usetokenonread" isn't set in the namespaces json.