-
Notifications
You must be signed in to change notification settings - Fork 825
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
missing cookie switch #775
Conversation
Hey strehle! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/155191148 The labels on this github issue will be updated when the story is started. |
We actually do need the Current User cookie. It is used by the session_management.html page together with RP javascripts in uaa-singular for our implementation of OIDC session management. It seems if any change is needed here, it is to add assertions about the presence of Current User cookie to relevant integration tests. |
…ookie_switch * 'bug/cookie_switch' of github.com:strehle/uaa: Send account cookies only if idp discovery switch is on
Merged via 3ae3e72 Thanks for the PR! |
see issue #774
This PR is a suggestion to omit the cookie.
The existance is required in mock tests but only in mocks not integration tests