Gerbil produces wrongly parsed logs #1437

New Issue

MrUnknownDE · 2026-04-05T19:28:24+02:00

MrUnknownDE commented

2026-04-05 19:28:24 +02:00

Originally created by @narbux on 5/29/2025

In the logs of Pangolin I noticed these lines appearing multiple times.

pangolin  | 2025-05-26T18:00:25.353Z [info]: Public key mismatch. Deleting old peer...
pangolin  | 2025-05-26T18:00:25.361Z [info]: Peer deleted successfully: {"0":"P","1":"e","2":"e","3":"r","4":" ","5":"r","6":"e","7":"m","8":"o","9":"v","10":"e","11":"d","12":" ","13":"s","14":"u","15":"c","16":"c","17":"e","18":"s","19":"s","20":"f","21":"u","22":"l","23":"l","24":"y"}
pangolin  | 2025-05-26T18:00:25.367Z [info]: Peer added successfully: {"0":"P","1":"e","2":"e","3":"r","4":" ","5":"a","6":"d","7":"d","8":"e","9":"d","10":" ","11":"s","12":"u","13":"c","14":"c","15":"e","16":"s","17":"s","18":"f","19":"u","20":"l","21":"l","22":"y"}

As you can see the text after 'Peer added/deleted successfully' is rendered as an object...

Which are probably the parsing of response.data.status in this code:
b467d6afa1/server/routers/gerbil/peers.ts (L27)
b467d6afa1/server/routers/gerbil/peers.ts (L47)

*Originally created by @narbux on 5/29/2025* In the logs of Pangolin I noticed these lines appearing multiple times. ``` pangolin | 2025-05-26T18:00:25.353Z [info]: Public key mismatch. Deleting old peer... pangolin | 2025-05-26T18:00:25.361Z [info]: Peer deleted successfully: {"0":"P","1":"e","2":"e","3":"r","4":" ","5":"r","6":"e","7":"m","8":"o","9":"v","10":"e","11":"d","12":" ","13":"s","14":"u","15":"c","16":"c","17":"e","18":"s","19":"s","20":"f","21":"u","22":"l","23":"l","24":"y"} pangolin | 2025-05-26T18:00:25.367Z [info]: Peer added successfully: {"0":"P","1":"e","2":"e","3":"r","4":" ","5":"a","6":"d","7":"d","8":"e","9":"d","10":" ","11":"s","12":"u","13":"c","14":"c","15":"e","16":"s","17":"s","18":"f","19":"u","20":"l","21":"l","22":"y"} ``` As you can see the text after 'Peer added/deleted successfully' is rendered as an object... Which are probably the parsing of `response.data.status` in this code: https://github.com/fosrl/pangolin/blob/b467d6afa198a4d77d6399d3418382b5081d7eae/server/routers/gerbil/peers.ts#L27 https://github.com/fosrl/pangolin/blob/b467d6afa198a4d77d6399d3418382b5081d7eae/server/routers/gerbil/peers.ts#L47

MrUnknownDE closed this issue

2026-04-05 19:28:24 +02:00

MrUnknownDE added the non-critical bug good first issue non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue labels 2026-04-05 19:28:26 +02:00

Sign in to join this conversation.

Branches Tags

main

dev

dependabot/npm_and_yarn/dev-minor-updates-b4e5d6b9c5

revert-2766-feature/systemd-install-instructions

dependabot/npm_and_yarn/prod-patch-updates-05702d39f2

dependabot/npm_and_yarn/next-16.2.1

dependabot/npm_and_yarn/recharts-3.8.1

alerting-rules

private-site-ha

dependabot/docker/docker/library/node-25-slim

ssh

delete-account

msg-delivery

org-only-idp

cicd

patch

site-targets-auto-login

No Label good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue good first issue non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug non-critical bug

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/pangolin#1437