Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Decrease snapshot timeout to avoid being disconnected #367

Merged
merged 9 commits into from
Jul 29, 2021

Conversation

bdraco
Copy link
Contributor

@bdraco bdraco commented Jul 27, 2021

  • iOS will terminate the connection if it does not respond within
    10 seconds, so we only allow 9 seconds to avoid this.

  • We no longer need to pause reading when taking a snapshot
    as we have a working timeout

@codecov
Copy link

codecov bot commented Jul 27, 2021

Codecov Report

Merging #367 (e123f5d) into dev (1124bab) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #367      +/-   ##
==========================================
- Coverage   91.01%   91.00%   -0.01%     
==========================================
  Files          20       20              
  Lines        1937     1935       -2     
  Branches      224      224              
==========================================
- Hits         1763     1761       -2     
  Misses        131      131              
  Partials       43       43              
Impacted Files Coverage Δ
pyhap/hap_protocol.py 100.00% <ø> (ø)
pyhap/hap_handler.py 81.23% <100.00%> (ø)

@bdraco bdraco force-pushed the decrease_timeout_avoid_disconnect branch from d128e4d to a48d1e2 Compare July 27, 2021 18:52
@bdraco bdraco marked this pull request as ready for review July 27, 2021 19:44
@ikalchev ikalchev merged commit af6c739 into ikalchev:dev Jul 29, 2021
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.

2 participants