Skip to content
This repository has been archived by the owner on Jul 15, 2021. It is now read-only.

[Fix #26] create multiple session cookies #85

Merged
merged 2 commits into from Jan 16, 2019

Conversation

paulczar
Copy link
Contributor

When used with auth systems that create large keys (UAA) storing
sessions as cookies can cause the cookie to be larger than
browsers like. This somewhat awkwardly seperates id_token and
refresh_token out into their own cookies.

You could also switch out to filesystem or redis for session storage
but that adds more complication for runtime than this super simple use
case really warrants IMHO.

Signed-off-by: Paul Czarkowski username.taken@gmail.com

@paulczar
Copy link
Contributor Author

this addresses #26

@paulczar
Copy link
Contributor Author

will rebase when/if the other PRs are merged in.

When used with auth systems that create large keys (UAA) storing
sessions as cookies can cause the cookie to be larger than
browsers like. This somewhat awkwardly seperates id_token and
refresh_token out into their own cookies.

You could also switch out to filesystem or redis for session storage
but that adds more complication for runtime than this super simple use
case really warrants IMHO.

Signed-off-by: Paul Czarkowski <username.taken@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants