-
-
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(logger): ignore session logging during setup #47521
Conversation
Or we wrap the code with an |
if we can ignore slow session logging during installation process, I could check this in |
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.
I think both makes sense.
d0f57b5
to
509e3f1
Compare
@nickvergessen please check this new way of fixing the issue. |
Signed-off-by: Maxence Lange <[email protected]>
509e3f1
to
9100b87
Compare
Thanks. Fixed, squashed and rebased |
/backport to stable30 |
/backport to stable29 |
(no idea if an urgent new release of 29 is needed) |
Confirmed that this fixes nextcloud/docker#2283 |
This breaks web based installations. While |
This fix a regression from #47105 that lock installation process on 29.0.5 (but no 30 😕 ) when using the web interface