search

WalletConnect / notify-server

notify.walletconnect.com
https://docs.walletconnect.com/web3inbox/about
MIT License
20 stars 6 forks source link

Providing the same address to /notify multiple times will return inconsistent results #313

Open chris13524 opened 10 months ago

chris13524 commented 10 months ago

Uncertain on current behavior, but if the same address is provided multiple times one of them may be returned in not_found. However, since they are the same account it would be best if the behavior was idempotent and returned sent for both?

chris13524 commented 10 months ago

Added a log which should detect this: https://github.com/WalletConnect/notify-server/blob/4fe3fd6ce2f5fefa1873e2293329919ed55d160e/src/services/public_http_server/handlers/notify_v1.rs#L226