-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
WIP: Add "ring" message logger to review last N messages on /admin/logging #18858
Conversation
Just FYI: there was an old issue for this: |
NOTE: this ring buffer does live within an instance ... for clustering this will be an issue ... |
What is clustering? |
What about adding a method to get the current content of a named provider? It could be empty if the provider does not support that (smtp for instance). Or it could be the full content of the log file for the file provider. Or the content of the rotated list of messages for this new provider. |
81387c5
to
abdabbd
Compare
I think it means multiple gitea instances running off the same databases/cache/filesystem with traffic load-balanced between them. Any data specific to a single instance is problematic in such a case, it must be in a shared databases/cache/filesystem. |
WIP: not ready yet. Signed-off-by: Andrew Thornton <art27@cantab.net>
abdabbd
to
b99bf5a
Compare
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 months. Thank you for your contributions. |
This is in progress for a while now, I'm closing it to avoid stale pull requests. Please reopen when it's ready for review. |
Add a ring message logger to make it possible to see the last N log messages from the admin settings screen.
Very much WIP. UI not ready etc.
Current hacky way to add in the messages logger is to add:
Signed-off-by: Andrew Thornton art27@cantab.net