Skip to content

Conversation

@keynslug
Copy link
Contributor

@keynslug keynslug commented Jun 23, 2025

Fixes EMQX-14429.

Release version: 5.10.1?

Summary

emqx/cowboy#13 includes fixes for "crash-on-shutdown" bug in cowboy_websocket_linger logic.

PR Checklist

  • For internal contributor: there is a jira ticket to track this change

  • The changes are covered with new or existing tests
    No new testcases: existing tests (namely, emqx_persistent_session_SUITE.persistence_disabled.ws test group) are now free of crash reports and "unknown message" warnings.

  • Change log for changes visible by users has been added to changes/ee/(feat|perf|fix|breaking)-<PR-id>.en.md files

  • Schema changes are backward compatible

keynslug added 3 commits June 23, 2025 14:37
That includes fixes for "crash-on-shutdown" bug in
`cowboy_websocket_linger` logic.
@keynslug keynslug marked this pull request as ready for review June 23, 2025 13:26
@keynslug keynslug requested a review from a team as a code owner June 23, 2025 13:26
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.77%. Comparing base (3eb6169) to head (4004bea).

Additional details and impacted files
@@               Coverage Diff               @@
##           release-510   #15416      +/-   ##
===============================================
+ Coverage        81.63%   81.77%   +0.14%     
===============================================
  Files             1117     1117              
  Lines            81559    81559              
===============================================
+ Hits             66579    66695     +116     
+ Misses           14980    14864     -116     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@keynslug keynslug merged commit 9770b63 into emqx:release-510 Jun 23, 2025
450 of 453 checks passed
@keynslug keynslug deleted the fix/EEC-1209/ws-shutdown-crash branch June 23, 2025 14:03
@emqxqa
Copy link

emqxqa commented Jul 7, 2025

TestExecution

1 similar comment
@emqxqa
Copy link

emqxqa commented Jul 7, 2025

TestExecution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants