portal detection misses non-standard portal status codes #36

Open
opened 2026-10-15 11:11:28 +08:00 by Breadway · 0 comments
Owner

Found: full codebase audit 2026-10-15, by reading src/status.rs is_portal_status.
Confidence: medium.
Location: src/status.rs:66-68.

Only 200/301/302/303/307/308/511 count as portal. A portal answering 404/403 with a login page plus blocked ping classifies DownNoNet, so the watcher reconnects pointlessly instead of telling the user to sign in.

Suggested fix: broaden portal-shaped codes (any non-204 2xx/3xx) or treat unexpected bodies as portal candidates.

Found: full codebase audit 2026-10-15, by reading src/status.rs is_portal_status. Confidence: medium. Location: src/status.rs:66-68. Only 200/301/302/303/307/308/511 count as portal. A portal answering 404/403 with a login page plus blocked ping classifies DownNoNet, so the watcher reconnects pointlessly instead of telling the user to sign in. Suggested fix: broaden portal-shaped codes (any non-204 2xx/3xx) or treat unexpected bodies as portal candidates.
Sign in to join this conversation.
No labels
bug
ci
enhancement
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
Breadway/breadcrumbs#36
No description provided.