Investigate Crash on Router restart #17

Closed
opened 2023-03-18 16:06:05 +00:00 by Firq · 5 comments
Firq commented 2023-03-18 16:06:05 +00:00 (Migrated from gitlab.neshweb.net)

Problem:

Since the DSL provider seemed to lose power and no requests were possible, it seems like the resulting timeouts were the source of the crash.
Feel free to investigate further

Proposed solution:

Implement error-handling if the responses time out

**Problem:** Since the DSL provider seemed to lose power and no requests were possible, it seems like the resulting timeouts were the source of the crash. Feel free to investigate further **Proposed solution:** Implement error-handling if the responses time out
Firq commented 2023-03-18 16:06:06 +00:00 (Migrated from gitlab.neshweb.net)

assigned to @Neshura

assigned to @Neshura
Neshura commented 2023-03-18 16:09:03 +00:00 (Migrated from gitlab.neshweb.net)

Issue is unrelated to Router restarts, also occasionally occurs without them. Cause is an expected field on the HTTP Request that is not present.

Fix should be easy (check HTTP Status code before parsing) but I'm lazy.

Issue is unrelated to Router restarts, also occasionally occurs without them. Cause is an expected field on the HTTP Request that is not present. Fix should be easy (check HTTP Status code before parsing) but I'm lazy.
Neshura commented 2023-03-21 09:41:45 +00:00 (Migrated from gitlab.neshweb.net)

mentioned in commit 2287e8da9d

mentioned in commit 2287e8da9d9668042d5fc429899156bb8fb1a905
Neshura commented 2023-03-27 09:12:13 +00:00 (Migrated from gitlab.neshweb.net)

Issue seems to be fixed, closing for now.

Issue seems to be fixed, closing for now.
Firq commented 2023-03-27 09:48:59 +00:00 (Migrated from gitlab.neshweb.net)

image

![image](/uploads/2dd29f2460098c34f037c9cf8c795a6d/image.png)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Neshura/cloudflare-dns-updater#17
No description provided.