-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Fix the logger that is imported for critical actions #31454
Conversation
84d2791
to
44c7065
Compare
44c7065
to
a307c9e
Compare
Unit tests are failing but I'm not sure it's caused by this PR: |
The "Set up php 7.4" command errors:
I assume this is a temporary bug with actions? But this here is not: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As this fixes a bug, we need to backport it, right?
yes |
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>
b484e84
to
3e0c42e
Compare
/backport to stable23 |
/backport to stable22 |
Add the import in 3e0c42e#diff-26f1e4805b227aa0936eaf6b5492ca8b166f852322243fb5d37b0d27e4c30f95R47 ? |
Currently it's just using the normal psr logger
Needed for nextcloud/groupfolders#1917