-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
Allow to have a unique user-id
across sessions
#1023
Allow to have a unique user-id
across sessions
#1023
Conversation
user-id
across sessions
🚲 PR staged at http://34.118.205.234 |
🚲 PR staged at http://34.118.205.234 |
🚲 PR staged at http://34.118.205.234 |
Just did my tests, ready for your review, thanks! Sorry for the confusion while I was doing my test in draft mode, if there is still confusion or your need clarification, feel free to ping me :) The default behavior is still the same, now if someone wants a unique user id across sessions, they could use this env var |
🚲 PR staged at http://34.118.205.234 |
1 similar comment
🚲 PR staged at http://34.118.205.234 |
…#1023) * ENABLE_UNIQUE_USER_ID_ACROSS_SESSIONS * Update middleware.go * fix ENABLE_UNIQUE_USER_ID_ACROSS_SESSIONS test with the value "true" * ENABLE_SINGLE_SHARED_SESSION
This introduces a new environment variable for the
frontend
app:ENABLE_SINGLE_SHARED_SESSION
. This allows to have a unique User Id across the sessions. The default behavior is one User Id per session.Fixing #615 + allows more scenarios to showcase:
redis
database is just in 1 cluster, see that the data are accessible fromcartservice
andfrontend
from different clusters.cartservice
andfrontend
from different clusters.If you set this environment variable to
"true"
on thefrontend
app, here is how to test this new behavior:user-id
at the bottom of the website (associated cookie) with the value: ``. If you open your internet browser now in InPrivate/Incognito mode, you will see the same value.Notes:
frontend
'sDeployment
manifest or for example run this command:kubectl set env deployment/frontend ENABLE_SINGLE_SHARED_SESSION="true"
.