sneak/neoirc
1
0
Fork 0
Code Issues 10 Pull Requests 3 Actions 30 Packages Projects Releases Wiki Activity

Replace HTTP status codes with IRC numeric replies in command handlers (closes #54) #55

Closed
clawbot wants to merge 2 commits from fix/irc-numeric-replies into main
pull from: fix/irc-numeric-replies
merge into: :main
Conversation 2 Commits 2 Files Changed 2
+240 -98
2 Commits
Author SHA1 Message Date
sneak 38d222f6a7 Merge branch 'main' into fix/irc-numeric-replies
check / check (push) Successful in 57s
Details
2026-03-09 22:12:43 +01:00
user 8d91ad852c Replace HTTP status codes with IRC numeric replies in command handlers (closes #54)
check / check (push) Successful in 2m17s
Details 
IRC command handlers now return proper IRC numeric reply codes per
RFC 1459/2812 instead of HTTP status codes:

- 401 ERR_NOSUCHNICK for unknown DM targets
- 403 ERR_NOSUCHCHANNEL for invalid/missing channels
- 411 ERR_NORECIPIENT for missing message recipients
- 412 ERR_NOTEXTTOSEND for missing message body
- 421 ERR_UNKNOWNCOMMAND for unknown/empty commands
- 431 ERR_NONICKNAMEGIVEN for missing nick in NICK command
- 432 ERR_ERRONEUSNICKNAME for invalid nick format
- 433 ERR_NICKNAMEINUSE for taken nicks
- 442 ERR_NOTONCHANNEL for non-member channel actions
- 461 ERR_NEEDMOREPARAMS for missing required parameters

Error responses use the IRC numeric format:
  {"command":"4xx","from":"server","to":"nick","body":["..."],"params":[...]}

HTTP status codes are now reserved for transport-level concerns:
- 400 for malformed HTTP requests (bad JSON)
- 401 for authentication failures
- 500 for internal server errors

Successful message sends changed from 201 to 200 since HTTP
status codes should not encode IRC-level semantics.
 2026-03-08 01:16:04 -08:00