Skip to content
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.

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

Continue to next packet if received broken packet #1260

Merged
merged 1 commit into from Oct 23, 2022

Conversation

ozraru
Copy link
Contributor

@ozraru ozraru commented Oct 19, 2022

Same as #1230 .
Sorry, I made a mistake and closed it.

#1166 has avoid panic but cause a silent fail.
This fixes it.
Broken packet should be ignored and continue to next packet.
This goroutine should not exit until close.

Broken packet should be ignored and continue to next packet.
This goroutine should not exit until close.
@FedorLap2006 FedorLap2006 self-assigned this Oct 19, 2022
@FedorLap2006 FedorLap2006 added fix Pull requests and issues related to bug fixes and structural inconsistencies low priority and removed low priority labels Oct 19, 2022
@FedorLap2006 FedorLap2006 merged commit 0c2b0ec into bwmarrin:master Oct 23, 2022
@FedorLap2006
Copy link
Collaborator

Thanks for the contribution!

LWCoder pushed a commit to AtTheTavern/discordgo that referenced this pull request Apr 11, 2023
Broken packet should be ignored and continue to next packet.
This goroutine should not exit until close.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Pull requests and issues related to bug fixes and structural inconsistencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants