Uninett / zinolib

Python library for zino
Apache License 2.0
1 stars 3 forks source link

Add support for keepalive #63

Closed hmpf closed 2 weeks ago

hmpf commented 3 weeks ago

Closes #62, probably

https://stackoverflow.com/questions/12248132/how-to-change-tcp-keepalive-timer-using-python-script

sonarcloud[bot] commented 3 weeks ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

github-actions[bot] commented 3 weeks ago

Test results

    3 files      3 suites   41s :stopwatch:   66 tests   66 :heavy_check_mark: 0 :zzz: 0 :x: 198 runs  198 :heavy_check_mark: 0 :zzz: 0 :x:

Results for commit 9e11dac5.

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 68.96552% with 9 lines in your changes missing coverage. Please review.

Project coverage is 70.23%. Comparing base (ddfbd8c) to head (aeabe20). Report is 11 commits behind head on main.

Files Patch % Lines
src/zinolib/utils.py 72.22% 5 Missing :warning:
src/zinolib/ritz.py 63.63% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #63 +/- ## ========================================== - Coverage 70.49% 70.23% -0.27% ========================================== Files 13 13 Lines 1427 1458 +31 ========================================== + Hits 1006 1024 +18 - Misses 421 434 +13 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

hmpf commented 2 weeks ago

After help with experimentation with curitz I have added keepalive to the protocol socket (on 8001) as well.