Skip to content
New issue

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

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

Already on GitHub? # to your account

Do not stack overflow if a TCP frame contains too many PUBLISH #738

Merged
merged 2 commits into from
Dec 9, 2017

Conversation

mcollina
Copy link
Member

@mcollina mcollina commented Dec 9, 2017

No description provided.

@codecov
Copy link

codecov bot commented Dec 9, 2017

Codecov Report

Merging #738 into master will increase coverage by 0.11%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #738      +/-   ##
==========================================
+ Coverage   93.33%   93.44%   +0.11%     
==========================================
  Files           8        8              
  Lines         720      748      +28     
  Branches      178      196      +18     
==========================================
+ Hits          672      699      +27     
- Misses         48       49       +1
Impacted Files Coverage Δ
lib/client.js 96.84% <100%> (+0.01%) ⬆️
lib/connect/index.js 93.26% <0%> (+0.96%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 21084f5...58a520b. Read the comment docs.

@mcollina mcollina merged commit 3323089 into master Dec 9, 2017
@mcollina mcollina deleted the fix-loads-of-packets branch December 9, 2017 21:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant