8000 fix: avoid writing messages after close and improve handshake by FrauElster · Pull Request #476 · coder/websocket · GitHub
[go: up one dir, main page]
Skip to content

fix: avoid writing messages after close and improve handshake #476

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Dec 4, 2024
Merged

fix: avoid writing messages after close and improve handshake #476

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
0daf07f
adds a lock on writeClose. closes #448
FrauElster Sep 5, 2024
87caf58
moves the writeLock to writeFrame to avoid deadlock
FrauElster Sep 6, 2024
e70e060
group closeSent with writeFrameMu
mafredri Sep 6, 2024
2ea151a
return close sent for all writes, remove sentinel error
mafredri Sep 6, 2024
c3613fc
fix: echo read error after close received
mafredri Sep 17, 2024
8a6704c
Merge branch 'master' into FrauElster/master
mafredri Nov 21, 2024
cc1c15a
fix: rewrite close handshake flow when initiated from other side
mafredri Nov 29, 2024
038d6d0
fix timeout
mafredri Nov 29, 2024
9cd2adf
adjust opclose err on closed
mafredri Nov 29, 2024
e8f5b51
move disconnected comment
mafredri Nov 29, 2024
e410eab
add mutex comment
mafredri Nov 29, 2024
930432f
comment prepareRead and readMu unlocking
mafredri Dec 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
group closeSent with writeFrameMu
  • Loading branch information
@mafredri
mafredri committed Sep 6, 2024
commit e70e06008b007314fb8637099110c7eb88fd343d
8 changes: 4 additions & 4 deletions conn.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,15 +68,15 @@ type Conn struct {
writeBuf []byte
writeHeaderBuf [8]byte
writeHeader header
closeSent bool

closeReadMu sync.Mutex
closeReadCtx context.Context
closeReadDone chan struct{}

closed chan struct{}
closeMu sync.Mutex
closing bool
closeSent bool
closed chan struct{}
closeMu sync.Mutex
closing bool

pingCounter atomic.Int64
activePingsMu sync.Mutex
Expand Down
0