Skip to content

Conversation

@ajey35
Copy link

@ajey35 ajey35 commented Jan 22, 2026

Enable Debug Logging by Default

This PR enables debug logging by default and fixes #59 broken log-level propagation.

Changes

  • Set default log level to debug in the Electron main process.
  • Pass the active logLevel to coinswap-worker.js.
  • Fix crash in taker:setupLogging caused by an undefined TakerClass.
  • Ensure worker logger uses the same log level as the main process.

Result

  • Debug logs enabled on fresh installs.
  • Consistent logging across main and worker processes.
  • Logging setup from the UI works correctly.

Files Changed

  • api1.js
  • coinswap-worker.js

Copy link
Collaborator

@keraliss keraliss left a comment

Choose a reason for hiding this comment

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

looks good, but we also need a button on the log page to show the debug logs, like we have for info, warning and error. can you add that as well? one dedicated space for debug logs?

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.

enable debug logging by default for the app

2 participants