bitcoin-sv / alert-system

Alert System Microservice
Other
3 stars 2 forks source link

[Bug]: Include libp2p topic name in database table #42

Closed galt-tr closed 8 months ago

galt-tr commented 9 months ago

What happened?

When switching topics with the same binary/database combination it would be ideal if I could share the same database and start/stop the server switching topics without the alerts being munged together.

We should include the topic name in the database table name to prevent collisions.

How can we reproduce this?

Run this:

$ ALERT_SYSTEM_ENVIRONMENT=testnet ./alert-system
# stop the server
$ ALERT_SYSTEM_ENVIRONMENT=stn ./alert-system

Server will just assume all alerts in the database are valid for this topic and environment if the keys are the same.

Search

Code of Conduct

Additional context

No response

galt-tr commented 8 months ago

This was fixed with 0.1.1