ntex-rs / ntex-amqp

AMQP 1.0 Server framework
Apache License 2.0
66 stars 15 forks source link

fix session end #17

Closed wpbrown closed 2 years ago

wpbrown commented 2 years ago

initiating side of session end needs to enter closing state so the end response is not replied to.

codecov[bot] commented 2 years ago

Codecov Report

Merging #17 (6685f0f) into master (e5fcc1f) will increase coverage by 0.35%. The diff coverage is 91.30%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   53.08%   53.44%   +0.35%     
==========================================
  Files          35       35              
  Lines        3805     3815      +10     
==========================================
+ Hits         2020     2039      +19     
+ Misses       1785     1776       -9     
Impacted Files Coverage Δ
src/session.rs 42.41% <83.33%> (+0.59%) :arrow_up:
src/connection.rs 71.19% <100.00%> (+6.75%) :arrow_up:
tests/test_server.rs 88.34% <100.00%> (+0.11%) :arrow_up:
src/server/service.rs 64.75% <0.00%> (-0.82%) :arrow_down:
src/dispatcher.rs 46.55% <0.00%> (-0.58%) :arrow_down:

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 e5fcc1f...6685f0f. Read the comment docs.

fafhrd91 commented 2 years ago

thanks!