Skip to content

fix: Log critical session renewal and logout paths - #40785

Merged
ChristophWurst merged 1 commit into
masterfrom
fix/user/log-logout-conditions
Oct 10, 2023
Merged

fix: Log critical session renewal and logout paths#40785
ChristophWurst merged 1 commit into
masterfrom
fix/user/log-logout-conditions

Conversation

@ChristophWurst

@ChristophWurstChristophWurst commented Oct 5, 2023•
edited
Loading

Copy link
Copy Markdown
Member

Summary

Extracted and extended the generic logging pieces of #40628.

TODO

  • Add logging

Checklist

@ChristophWurstChristophWurst self-assigned this Oct 5, 2023
Altahrim
Altahrim reviewed Oct 5, 2023
View reviewed changes
Comment threadlib/private/User/Session.php
Altahrim
Altahrim reviewed Oct 5, 2023
View reviewed changes
Comment threadlib/private/User/Session.php Outdated
Altahrim
Altahrim reviewed Oct 5, 2023
View reviewed changes
Comment threadlib/private/User/Session.php
@ChristophWurst

ChristophWurst commented Oct 5, 2023

Copy link
Copy Markdown
MemberAuthor

/backport to stable27

@ChristophWurst

ChristophWurst commented Oct 5, 2023

Copy link
Copy Markdown
MemberAuthor

/backport to stable26

come-nc
come-nc reviewed Oct 9, 2023
View reviewed changes
Comment threadlib/private/User/Session.php Outdated
@ChristophWurst
fix: Log critical session renewal and logout paths
f398d0b
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst
ChristophWurstforce-pushed the fix/user/log-logout-conditions branch from 4a02c5a to f398d0bCompareOctober 9, 2023 11:21
@ChristophWurst
ChristophWurst merged commit 8c2b9b3 into masterOct 10, 2023
@ChristophWurst
ChristophWurst deleted the fix/user/log-logout-conditions branch October 10, 2023 09:26
This was referenced Oct 10, 2023
Merged
Merged
@ChristophWurstChristophWurst mentioned this pull request Oct 11, 2023
Merged
4 tasks
@solracsfsolracsf mentioned this pull request Oct 29, 2023
Closed
8 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AltahrimAltahrimAltahrim approved these changes
@come-nccome-nccome-nc approved these changes
@nickvergessennickvergessenAwaiting requested review from nickvergessen

Assignees

@ChristophWurstChristophWurst

Labels

3. to reviewWaiting for reviewsbugfeature: authenticationfeature: users and groups

Projects

💌 📅 👥 Groupware team
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ChristophWurst@nickvergessen@Altahrim@come-nc