Skip to content

feat(onvif-events): Restart event requests after ECONNRESET error#320

Merged
agsh merged 4 commits into
masterfrom
feat-restart-event-request-after-connection-error
May 19, 2024
Merged

feat(onvif-events): Restart event requests after ECONNRESET error#320
agsh merged 4 commits into
masterfrom
feat-restart-event-request-after-connection-error

Conversation

@agsh
Copy link
Copy Markdown
Owner

@agsh agsh commented May 19, 2024

This is a compilation, refactoring and tests for two PRs about reconnecting if the connection to the device fails: #314 #319

@agsh agsh requested a review from RogerHardiman May 19, 2024 00:03
@agsh agsh self-assigned this May 19, 2024
@agsh agsh force-pushed the feat-restart-event-request-after-connection-error branch from f865285 to e1dbc71 Compare May 19, 2024 00:07
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 19, 2024

Deploying onvif with  Cloudflare Pages  Cloudflare Pages

Latest commit: d04eae0
Status: ✅  Deploy successful!
Preview URL: https://792d14f7.onvif.pages.dev
Branch Preview URL: https://feat-restart-event-request-a.onvif.pages.dev

View logs

@agsh agsh force-pushed the feat-restart-event-request-after-connection-error branch from e1dbc71 to 7903189 Compare May 19, 2024 14:22
@HarryEMartland
Copy link
Copy Markdown
Contributor

Yep that is looking good to me and is reconnecting to my doorbell after a restart.

Ill get an MR into Shinobi once the new version is available.

Thanks for your help with this!

@agsh agsh merged commit a325700 into master May 19, 2024
@agsh
Copy link
Copy Markdown
Owner Author

agsh commented May 19, 2024

@HarryEMartland You're welcome! 😉 Release is ready: https://github.com/agsh/onvif/actions/runs/9150337468

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants