Skip to content

docs(config): document session_lifetime semantics and interactions#58794

Open
joshtrichards wants to merge 2 commits into
masterfrom
jtr/docs-config-session-lifetime
Open

docs(config): document session_lifetime semantics and interactions#58794
joshtrichards wants to merge 2 commits into
masterfrom
jtr/docs-config-session-lifetime

Conversation

@joshtrichards

@joshtrichards joshtrichards commented Mar 8, 2026

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

🧹

Related PR: #58792

TODO

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

…nteractions

Signed-off-by: Josh <josh.t.richards@gmail.com>
Comment thread config/config.sample.php Outdated
Comment on lines +345 to +346
* Lifetime of sessions created without selecting "Remember me", after
* inactivity, in seconds.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This does not sound true to me, the value is also used for sessions where remember me is selected.
It’s just that at log out you get logged in by the remember me cookie, no?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Indeed. Adjusted.

This was referenced Jun 5, 2026
@susnux susnux added the community pull requests from community label Jun 9, 2026
@susnux susnux modified the milestones: Nextcloud 34.0.1, Nextcloud 35 Jun 9, 2026
Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards changed the title docs(config): clarify session_lifetime semantics and related expiry interactions docs(config): document session_lifetime semantics and interactions Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews community pull requests from community enhancement feature: authentication

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants