-
Notifications
You must be signed in to change notification settings - Fork 6
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
jwt settings #122
jwt settings #122
Conversation
Instead show a 403 error, that looks better
egi_notebooks_hub/d4science.py
Outdated
@@ -188,11 +188,12 @@ async def get_resources(self, access_token): | |||
async def authenticate(self, handler, data=None): | |||
# first get authorized upstream | |||
user_data = await super().authenticate(handler, data) | |||
context = quote_plus(getattr(self, "d4science_context", None)) | |||
context = getattr(self, "d4science_context", None) |
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.
i guess this will need to be updated when this is merged #117
The code is working (except the known missing parts, which can go in another PR). So it could be approved. What about the D4Science part? Is this PR safe to merge? |
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.
tested by @valtri, Looks good to go!
Summary
Make the API wrapper configurable so it runs beyond the simple local scenario
Related issue :