ipfs / go-graphsync

Initial Implementation Of GraphSync Wire Protocol
Other
100 stars 38 forks source link

Fix shutdown bug in #412 #422

Closed hannahhoward closed 1 year ago

hannahhoward commented 1 year ago

Goals

This resolves the problem created by #412, without reverting the change that created a memory error.

Essentially the problem is the queue is shut down without being removed from the peer manager, which continues to send it messages, that have no effect. This change adds a hook so that the queue itself can notify the peer manager it's been shutdown.