Skip to content

feat: add get node logs p2p handler#1199

Open
ndrpp wants to merge 2 commits intomainfrom
feat/281-get-node-logs-in-dashboard-with-p2p
Open

feat: add get node logs p2p handler#1199
ndrpp wants to merge 2 commits intomainfrom
feat/281-get-node-logs-in-dashboard-with-p2p

Conversation

@ndrpp
Copy link
Member

@ndrpp ndrpp commented Feb 11, 2026

Fixes # .

Changes proposed in this PR:

  • add admin get logs p2p handler
  • allow up to 1k log lines to be downloaded from typesense/elastic

@ndrpp ndrpp marked this pull request as ready for review February 11, 2026 09:03
@giurgiur99
Copy link
Contributor

What happens if LOG_DB is false and logs are not saved in db? Can we at leas download the exceptions from /logs dir?

@ndrpp
Copy link
Member Author

ndrpp commented Feb 11, 2026

What happens if LOG_DB is false and logs are not saved in db? Can we at leas download the exceptions from /logs dir?

If LOG_DB is false it will return an empty array. Should we download the exceptions?

Copy link
Member

@alexcos20 alexcos20 left a comment

Choose a reason for hiding this comment

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

why not use the new handler in src/components/httpRoutes/logs.ts ?

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.

3 participants