Closed mschneider82 closed 2 years ago
Thanks @mschneider82! Could you please turn on race mode for testing to prove this works (as it looks it does)?
I have added a nil check for atomic.Value with default time.Time{} (no deadline timeout) because TestPassthrough failed. In case no deadline was set
@pires i have tested it local with -race on. i will add -race to github actions
done, ready for review & merge
Fixed the race by using atomic.Value instead of unprotected time.Time
It was nessasarry to restructure the Conn struct because of this:
struct of size 80 could be 72 (fieldalignment)