Skip to content

fix: race condition in multiplex/writer - #132

Merged
caarlos0 merged 4 commits into
mainfrom
race
Dec 22, 2022
Merged

caarlos0 merged 4 commits into
mainfrom
race

Conversation

@caarlos0

@caarlos0 caarlos0 commented Dec 22, 2022 •

Copy link
Copy Markdown
Contributor

bytes.Buffer are not thread safe, so we need to wrap the writes/reads/etc with a mutex.

also improved the first thingy while I'm at it.

Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 self-assigned this Dec 22, 2022
@caarlos0 caarlos0 added the enhancement New feature or request label Dec 22, 2022
@caarlos0
caarlos0 marked this pull request as ready for review December 22, 2022 14:26
Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@caarlos0
caarlos0 merged commit bed3320 into main Dec 22, 2022
@caarlos0
caarlos0 deleted the race branch December 22, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

⌂ 换网址