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

fix(liveness): correct websocket retry logic #2634

Merged
merged 5 commits into from
Nov 11, 2023

Conversation

tjleing
Copy link
Contributor

@tjleing tjleing commented Nov 10, 2023

changes since #2615:

  • fix the bug where only offset device times were functional
  • add unit tests to cover the scenarios

manual tests:

  • test with correct device time
  • test with incorrect device time

@tjleing tjleing requested a review from a team as a code owner November 10, 2023 17:42
@codecov-commenter
Copy link

Codecov Report

Merging #2634 (e586c5b) into main (13d7fa0) will decrease coverage by 0.01%.
The diff coverage is 75.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main    #2634      +/-   ##
==========================================
- Coverage   41.78%   41.77%   -0.01%     
==========================================
  Files         901      901              
  Lines       28965    28959       -6     
  Branches     4102     4101       -1     
==========================================
- Hits        12102    12098       -4     
+ Misses      15536    15534       -2     
  Partials     1327     1327              

gpanshu
gpanshu previously approved these changes Nov 10, 2023
@tjleing tjleing merged commit ecb8021 into main Nov 11, 2023
5 checks passed
@tjleing tjleing deleted the liveness-web-socket-expiration-retry-fix branch November 11, 2023 03:54
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.

5 participants