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

include access token in client requests, if defined #133

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

minrk
Copy link
Contributor

@minrk minrk commented Apr 19, 2024

This allows h5web to work in a cross-site jupyterlab deployments (where backend and static pages are on different domains) and JupyterHub 4.1, which has stricter XSRF rules, and matches JupyterLab's own API requests made with ServerConnection.

Viewing hdf5 files results in a 403 error in these two scenarios without this patch.

this allows h5web to work in CORS jupyterlab deployments
and JupyterHub 4.1 which has stricter XSRF rules
@t20100
Copy link
Member

t20100 commented Apr 19, 2024

I just encounter this yesterday and was about to look into it, thanks for the PR!

Copy link
Contributor

@axelboc axelboc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks!

@axelboc axelboc merged commit 50e2a66 into silx-kit:main Apr 19, 2024
1 check passed
@rcthomas rcthomas mentioned this pull request Apr 19, 2024
@minrk minrk deleted the token branch April 20, 2024 17:08
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