Skip to content

[stable22] Fix the logger that is imported for critical actions - #31541

Merged
blizzz merged 1 commit into
stable22from
backport/31454/stable22
Apr 14, 2022
Merged

[stable22] Fix the logger that is imported for critical actions#31541
blizzz merged 1 commit into
stable22from
backport/31454/stable22

Conversation

@backportbot-nextcloud

Copy link
Copy Markdown

backport of #31454

This use a new interface AuditLogger that makes it easier to
seperate the roles of the various loggers and enforce that we
don't use the wrong one by mistake.
Signed-off-by: Carl Schwan <carl@carlschwan.eu>

@CarlSchwanCarlSchwan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

But I don't think it's important enough to backport it between the two rc

@szaimen

Copy link
Copy Markdown
Contributor

This probably needs #31454 (comment), too

@blizzzblizzz mentioned this pull request Mar 16, 2022
@blizzzblizzz mentioned this pull request Apr 13, 2022
1 task
@blizzzblizzz added the 3. to review Waiting for reviews label Apr 14, 2022
@blizzz
blizzz merged commit 3539f61 into stable22Apr 14, 2022
@blizzz
blizzz deleted the backport/31454/stable22 branch April 14, 2022 10:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@szaimen@blizzz@CarlSchwan