Add Canary wallet monitoring app#984
Conversation
|
@tehelsper , do you have any time soon to take a look at this? |
|
I should be able to get to it in the next release. I've got a couple quick comments. Thanks!
|
|
I can update this to v1.4.0, but I’d prefer not to do that yet without testing it on a real myNode device first. v1.4.0 adds new self-hosted behavior around local Mempool integration, and I haven’t had a chance to validate that path on actual myNode hardware yet. ntfy should be fine since it’s configured in Canary’s settings and defaults to ntfy.sh, but local Mempool support on myNode likely deserves a proper follow-up so Canary can detect and use the local instance cleanly. Since I have tested the integration itself on a real myNode box with the current Canary build, I think the safer path for this PR is to keep it on the tested version for now, and then do a follow-up bump to v1.4.0 once I can verify the myNode-specific Mempool behavior properly. |
|
Retested Canary on a fresh myNode machine and updated this branch to v1.4.0. Validation summary:
Changes included in this update:
This has now been pushed to the PR branch for review. |
| # Give backend time to start | ||
| ExecStartPre=/bin/sleep 2 | ||
| # Start frontend container (main process) | ||
| ExecStart=/usr/bin/docker run --rm --name canary-frontend --network host -e API_URL=http://127.0.0.1:3010 -e PORT=3005 schjonhaug/canary-frontend:v1.4.0 |
There was a problem hiding this comment.
How is this 3010 port being used? That's already taken by RTL.
There was a problem hiding this comment.
This also specifically uses the version tagged 1.4.0, it could use latest if the install process always:
- Removes old versions
- Installs new version
- Tags new version as latest
The install_albyhub.sh script might offer a good pattern for how to do this.
| docker rm canary canary-frontend 2>/dev/null || true | ||
|
|
||
| echo "Removing Docker images..." | ||
| docker rmi schjonhaug/canary-backend:v1.3.0 2>/dev/null || true |
There was a problem hiding this comment.
This still removes the old version. Look at install_albyhub.sh lines 24-25 to see if they can help remove old images generically rather than a specific version.
| set -e | ||
|
|
||
| echo "Pulling Canary Docker images..." | ||
| docker pull schjonhaug/canary-backend:v1.4.0 |
There was a problem hiding this comment.
Try using {VERSION} rather than the specific tag here, that way the app can be updated just by updating the JSON file that manages the app.
|
Thanks for the feedback, I've addressed the requested changes and pushed the updates to the PR branch. What changed:
The PR branch has been updated and is ready for another look. |
Description
Adds Canary - a Bitcoin wallet monitoring and early warning system for cold storage.
Features:
Canary connects to the local Electrum server (electrs) to monitor wallets without requiring private keys.
Checklist
List of test device(s)