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

Store state data in '__session' cookie instead of 'state' cookie #852

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tomcurran
Copy link
Contributor

@tomcurran tomcurran commented Apr 1, 2021

Firebase Functions seems to strip away any cookies not named '__session'. To get the Instagram sample to work I had to use the __session cookie instead of state

this should also fix

…information. Firebase Functions seems to strip away any cookies not named '__session'
@google-cla google-cla bot added the cla: yes label Apr 1, 2021
@tomcurran tomcurran changed the title Used '__session' cookie instead of 'state' cookie to store state data Store state data in '__session' cookie instead of 'state' cookie Apr 2, 2021
@samtstern samtstern requested a review from jhuleatt April 6, 2021 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant