-
Notifications
You must be signed in to change notification settings - Fork 37
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
enforce write only access to folder http endpoint #1109
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
bc875fc
to
c3a00ad
Compare
c3a00ad
to
e829646
Compare
fb2180a
to
4da0c4d
Compare
22cb540
to
4cf2141
Compare
4da0c4d
to
1722452
Compare
4cf2141
to
314dcb3
Compare
1722452
to
03640e4
Compare
314dcb3
to
9794d5a
Compare
# Conflicts: # runhouse/servers/http/http_utils.py
be625cd
to
dd1312d
Compare
9794d5a
to
4ad4f88
Compare
4ad4f88
to
ca83d03
Compare
# Conflicts: # runhouse/servers/http/http_server.py # runhouse/servers/http/http_utils.py
runhouse/servers/http/auth.py
Outdated
@@ -75,6 +75,7 @@ def clear_cache(self, token: str = None): | |||
async def averify_cluster_access( | |||
cluster_uri: str, | |||
token: str, | |||
write_only_access: bool = False, |
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.
instead of a bool here, can just do access_level_required
and pass ResourceAccess.WRITE
then below, can do return if access_level_required
cluster_access_level == access_level_required
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.
yea good call
ca83d03
to
bbd72e7
Compare
# Conflicts: # runhouse/servers/http/http_server.py # runhouse/servers/http/http_utils.py
bbd72e7
to
e5e16e7
Compare
# Conflicts: # runhouse/servers/http/http_server.py # runhouse/servers/http/http_utils.py
bfe2bc9
to
fbe82a6
Compare
No description provided.