aboutsummaryrefslogtreecommitdiffstats
path: root/commands
diff options
context:
space:
mode:
authorMichael Muré <batolettre@gmail.com>2022-06-05 15:01:08 +0200
committerMichael Muré <batolettre@gmail.com>2022-06-05 15:13:49 +0200
commit7348fb9edb68ca9142f5d87673da48cef733b3d3 (patch)
tree9d4667ecff1f40491d5825e60293e5c85a725104 /commands
parent96327c3371ca762d906209c6114092bbf552c0f4 (diff)
downloadgit-bug-7348fb9edb68ca9142f5d87673da48cef733b3d3.tar.gz
github: fix data race when closing event channel
I believe the issue was twofold: When done importing, the calling context is likely still valid, so if the output channel is not read enough and reach capacity, some event producer down the line can be blocked trying to send in that channel. When closing it, this send is still trying to proceed, which is illegal in go. In rateLimitHandlerClient, there was a need to 2 different type of output channel: core.ExportResult and ImportEvent. To do so, the previous code was using a single channel type RateLimitingEvent and a series of goroutines to read/cast/send to the final channel. This could result in more async goroutine being stuck trying to send in an at-capacity channel. Instead, the code now use a simple synchronous callback to directly push to the final output channel. No concurrency needed anymore and the code is simpler. Any of those fixes could have resolved the data race, but both fixes is more correct.
Diffstat (limited to 'commands')
0 files changed, 0 insertions, 0 deletions