Skip to content

Conversation

@blizzz
Copy link
Member

@blizzz blizzz commented May 25, 2022

- e.g. enables it for SAML backend

Signed-off-by: Arthur Schiwon <[email protected]>
@Ornanovitch
Copy link
Contributor

Ornanovitch commented May 25, 2022

Works well with NC24: now I get the three missing cookies with user_saml (nc_session_id, nc_token and nc_username), and they all expire in 15 days so everything seems fine! Thanks a lot 👍

Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works also with manually deleting the session from the php save path.

@juliusknorr
Copy link
Member

Failure unrelated

@juliusknorr juliusknorr merged commit ac79137 into master May 27, 2022
@juliusknorr juliusknorr deleted the fix/noid/allow-rememberme-token-apache-auth-backend branch May 27, 2022 15:21
@blizzz
Copy link
Member Author

blizzz commented May 31, 2022

/backport to stable24

@blizzz
Copy link
Member Author

blizzz commented May 31, 2022

/backport to stable23

@blizzz
Copy link
Member Author

blizzz commented May 31, 2022

/backport to stable22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remember_login cookie is not created

5 participants