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

Result.as must also copy flash, session and cookies data #9177

Conversation

Projects
None yet
3 participants
@marcospereira
Copy link
Member

commented Apr 8, 2019

Fixes #9176.

@mkurz

mkurz approved these changes Apr 9, 2019

Copy link
Member

left a comment

👍 I think I also bumped into this issue once but forgot to open a PR for it

@mergify mergify bot merged commit 557eb9e into playframework:master Apr 9, 2019

3 checks passed

Mergify — Summary 2 rules match
Details
Travis CI - Pull Request Build Passed
Details
typesafe-cla-validator All users have signed the CLA
Details

@marcospereira marcospereira deleted the marcospereira:fix-9176-result-as-should-copy-flash-and-session branch Apr 10, 2019

@mkurz

This comment has been minimized.

Copy link
Member

commented Apr 11, 2019

@dwijnand 🤖 Please backport as well, thanks!

dwijnand added a commit that referenced this pull request Apr 11, 2019

@dwijnand

This comment has been minimized.

Copy link
Member

commented Apr 11, 2019

2.7.x 55cf002

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.