bytebeamio / rumqtt

The MQTT ecosystem in rust
Apache License 2.0
1.53k stars 234 forks source link

feat: add MQTT v5 connection `session_expiry_interval` support #854

Closed xiaocq2001 closed 1 month ago

xiaocq2001 commented 2 months ago

Fixing issue https://github.com/bytebeamio/rumqtt/issues/853

Type of change

New feature (non-breaking change which adds functionality)

Checklist:

xiaocq2001 commented 1 month ago

Thanks for the PR! can you please also mention the changes in CHANGELOG as well?

Updated.

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 9072384592

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
rumqttc/src/v5/mod.rs 0 13 0.0%
<!-- Total: 0 13 0.0% -->
Files with Coverage Reduction New Missed Lines %
rumqttd/src/segments/mod.rs 7 97.34%
rumqttc/src/state.rs 42 78.83%
rumqttc/src/v5/state.rs 58 67.56%
<!-- Total: 107 -->
Totals Coverage Status
Change from base Build 8762117251: 0.1%
Covered Lines: 5983
Relevant Lines: 16393

💛 - Coveralls
xiaocq2001 commented 1 month ago

Moved the code to apply broker returned session_expiry_interval here, please check.