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

feat: persistent session at application level #1822

Merged
merged 1 commit into from
Apr 25, 2022

Conversation

remisultan
Copy link
Contributor

@remisultan remisultan commented Apr 21, 2022

fixes gravitee-io/issues#7526

This PR brings session persistence configuration at application level.

How to Test

First case : Inherited Configuration

image

  • Login to AM console
  • Go to Application > Your App > Settings > Cookie Settings
  • Configure your Cookie Settings with Inherited Configuration
  • Initiate the login flow (/domain/oauth/authorize) and login normally
  • Once logged in close your browser entirely
  • Initiate the login flow (/domain/oauth/authorize

Result: You should be able to be redirected automatically without a second login

Second case: Non persisted session

Screenshot 2022-04-21 at 23 34 11

  • Login to AM console
  • Go to Application > Your App > Settings > Cookie Settings
  • Configure your Cookie Settings with Persistent session to false
  • Initiate the login flow (/domain/oauth/authorize) and login normally
  • Once logged in close your browser entirely
  • Initiate the login flow (/domain/oauth/authorize) and login normally

Result: You should be redirected to /login

Third case: Persisted session

image

  • Login to AM console
  • Go to Application > Your App > Settings > Cookie Settings
  • Configure your Cookie Settings with Persistent session to true
  • Initiate the login flow (/domain/oauth/authorize) and login normally
  • Once logged in close your browser entirely
  • Initiate the login flow (/domain/oauth/authorize

Result: You should be able to be redirected automatically without a second login

@remisultan remisultan force-pushed the issues/#7526-peristent-session-mode branch 2 times, most recently from b3a9bb0 to 4520188 Compare April 21, 2022 21:54
@remisultan remisultan force-pushed the issues/#7526-peristent-session-mode branch from 3ead7a1 to e6aaefe Compare April 25, 2022 15:49
@tcompiegne tcompiegne force-pushed the issues/#7526-peristent-session-mode branch from e6aaefe to 5d54862 Compare April 25, 2022 16:02
@tcompiegne tcompiegne merged commit a5c5e25 into 3.10.x Apr 25, 2022
@tcompiegne tcompiegne deleted the issues/#7526-peristent-session-mode branch April 25, 2022 16:07
@tiagoros tiagoros mentioned this pull request Sep 21, 2022
This was referenced Oct 31, 2022
@ThibaudAV ThibaudAV mentioned this pull request Sep 27, 2024
@mergify mergify bot mentioned this pull request Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants