Make raven-go Wait() thread safe

getsentry / raven-go

Sentry client in Go

https://sentry.io

BSD 3-Clause "New" or "Revised" License

561 stars 148 forks source link

Make raven-go Wait() thread safe #238

Open jayapriya90 opened 5 years ago

jayapriya90 commented 5 years ago

Looks like raven-go's Wait() is not thread safe (https://github.com/getsentry/raven-go/blob/master/client.go#L875). When two go-routines concurrently try to mutate the state of sentry client, running into WaitGroup is reused before previous Wait has returned error