littlefox/neokom
Watch
1
2
Fork
You've already forked neokom
 1
Code Issues 8 Pull requests Projects Releases 12 Packages Activity Actions

Say why when the relay refuses the connection #142

Merged
littlefox merged 1 commit from issue-133-relay-refusal into main 2026-10-19 08:28:20 +00:00
Conversation 0 Commits 1 Files changed 10 +196 -40
littlefox commented 2026-10-19 07:57:56 +00:00
Owner
Copy link

Closes #133

The relay accepts a connection it won't serve and closes it with 4004/4005/4006; the login says what happened (full for your address, full, page not allowed) instead of 'check the address'.

Verified: relay integration tests (per-address, total, forwarded-for, origins), and the new e2e relay.spec (full relay, disallowed origin, no relay) plus tls.spec in Chromium.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XJW2ocwLunyaLiUwCBHEP8

Closes #133 The relay accepts a connection it won't serve and closes it with 4004/4005/4006; the login says what happened (full for your address, full, page not allowed) instead of 'check the address'. Verified: relay integration tests (per-address, total, forwarded-for, origins), and the new e2e relay.spec (full relay, disallowed origin, no relay) plus tls.spec in Chromium. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01XJW2ocwLunyaLiUwCBHEP8
Say why when the relay refuses the connection
All checks were successful
ci / check (push) Successful in 10m40s
Details
ci / desktop (push) Successful in 16m19s
Details
ci / e2e (push) Successful in 29m51s
Details
b4f2474819 
A browser only sees 'connection failed' for a refused WebSocket
handshake, so a full relay or a disallowed page showed 'Kunde inte nå
servern, kontrollera adressen', which sent people looking in the wrong
place. The relay now accepts such connections and closes them at once
with a code (4004 full for this address, 4005 full, 4006 origin not
allowed), and the login says what happened.

Refs #133

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XJW2ocwLunyaLiUwCBHEP8
littlefox referenced this pull request from a commit 2026-10-19 21:34:00 +00:00
Release 0.4.0
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
area/ci
Forgejo Actions, runner, tests

  
area/desktop
Desktop build

  
area/packaging
Arch, Debian, Fedora, NixOS packages

  
area/protocol
kom-proto, wire format, charset

  
area/reading
Unread tracking, reading order, review

  
area/ui
Dioxus app, layout, interaction

  
area/web
Web build and relay

  
area/writing
Composing and sending texts

  
kind/bug
Something is broken

  
kind/feature
New functionality

  
kind/infra
Tooling, tests, CI, dev environment

  
kind/request
requests for features or changes from users

  
needs/user
Waiting for a decision or action from littlefox

  
parked
Kept for reference; no work planned until the user says so

  
perf
Run the perf workflow and its budgets on this pull request (#287)

  
security
Security-relevant

  
upstream
A proposal for Protocol A or lyskomd, for the LysKOM development group

No labels
area/ci
area/desktop
area/packaging
area/protocol
area/reading
area/ui
area/web
area/writing
kind/bug
kind/feature
kind/infra
kind/request
needs/user
parked
perf
security
upstream
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
littlefox/neokom!142
No description provided.