-
Notifications
You must be signed in to change notification settings - Fork 683
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
Avoid restoring of flash and session multiple times per request #1181
Avoid restoring of flash and session multiple times per request #1181
Conversation
…ormance This is a better fix for stale sessions made in LH issue #1872.
…ion context This ensures that flash and session are resolved only once per request invocation.
a653565
to
73bb5fd
Compare
@xael-fry It seems to be a good PR. Recommended to merge. |
Merged and pushed in master |
Looking at changed files this doesn't look like it does anything other than adding an unused import? Maybe you forgot to merge something or something dropped along the line? Or does github maybe just not show it? |
Yes, the diff looks strange.
|
This fixes a regression introduced somewhere in Play 1.3
Session and flash will be restored only once per request (it is at least 3 times currently)
Depends on PR #1180, should be merged after it