fission-codes / go-car-mirror

Generic Go implementation of the CAR Mirror protocol
Apache License 2.0
4 stars 0 forks source link

Resolve potential concurrency issues around Orchestrator/Notify #45

Open softwareplumber opened 1 year ago

softwareplumber commented 1 year ago

In an Orchestrator, Notify should be the only method that mutates state.

At the moment Notify will perform conditional state updates in a way that is not always completely thread safe. We can't completely rely on the mutex in the SharedFlags object to achieve this for us.

ETA: 2023-10-31