Skip to content

Conversation

thgO-O
Copy link
Collaborator

@thgO-O thgO-O commented Sep 11, 2025

Changes

  • Implement server log viewer in the admin area.

  • Introduce Serilog with console and file sinks.

  • Add PluginBuilderOptions to centralize logging-related configuration.

Screenshot 2025-09-10 at 22 50 43

@thgO-O thgO-O requested a review from rockstardev September 11, 2025 11:58
@rockstardev rockstardev self-assigned this Sep 11, 2025
Copy link
Member

@rockstardev rockstardev left a comment

Choose a reason for hiding this comment

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

We discussed this during the meeting and agreed to review how logging is implemented in BTCPay Server, then apply that pattern to Plugin Builder (use the EXACT same env variables and such). There are two key areas to look at:

It's a bit unclear why both the built-in logging and Serilog are used – maybe @NicolasDorier can clarify that (I'll ping him).

Once we bring this logging pattern into Plugin Builder, we can port the improvements back into BTCPay Server. One immediate improvement we discussed is adding a config parameter to preserve logs for more than 1 day (currently, this is hardcoded in BTCPay Server).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants