-
Notifications
You must be signed in to change notification settings - Fork 242
Closed
Description
In this change:
98890dc
We moved to use the flat POST /subscriptions route on EthConnect, rather than requiring an ABI to be deployed to EthConnect first.
However, we seem to have missed specifying the name when we create it - so this check fails on every startup, and we're creating duplicate subscriptions for the same event:
firefly/internal/blockchain/ethereum/eventstream.go
Lines 177 to 185 in a435374
| for _, s := range existingSubs { | |
| if s.Name == subName || | |
| /* Check for the plain name we used to use originally, before adding uniqueness qualifier. | |
| If one of these very early environments needed a new subscription, the existing one would need to | |
| be deleted manually. */ | |
| s.Name == abi.Name { | |
| sub = s | |
| } | |
| } |
Metadata
Metadata
Assignees
Labels
No labels