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

feat: expose cookies and headers #501

Merged
merged 2 commits into from Feb 16, 2024

Conversation

iisakkirotko
Copy link
Collaborator

@iisakkirotko iisakkirotko commented Feb 13, 2024

Both cookies and headers are exposed as reactive variables, typed Optional[Dict[str, str]]

Fixes #263.

Edit: Note that cookies / headers can't be set via the reactive variables, but will be updated if kernel reconnects

Copy link
Collaborator Author

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@iisakkirotko iisakkirotko force-pushed the 02-13-feat_expose_cookies_and_headers branch from 0188148 to 414816a Compare February 13, 2024 13:37
@iisakkirotko iisakkirotko force-pushed the 02-13-feat_expose_cookies_and_headers branch 3 times, most recently from ea70e15 to 0ed15e5 Compare February 16, 2024 14:15
Both cookies and headers are exposed as reactive variables, typed `Optional[Dict[str, str]]`

Fixes #263.
@iisakkirotko iisakkirotko force-pushed the 02-13-feat_expose_cookies_and_headers branch 2 times, most recently from 56e5bfb to ae9273c Compare February 16, 2024 14:25
@iisakkirotko iisakkirotko force-pushed the 02-13-feat_expose_cookies_and_headers branch from ae9273c to abe6e22 Compare February 16, 2024 14:25
@maartenbreddels maartenbreddels merged commit bac204a into master Feb 16, 2024
47 checks passed
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.

Feature Request: Access to the request object for cookies and headers
2 participants