Skip to content

Add error handling audit for src/server.js#131

Merged
levibliz merged 2 commits into
RiftCore00:mainfrom
levoski1:issue-7-error-handling-server
Jun 13, 2026
Merged

Add error handling audit for src/server.js#131
levibliz merged 2 commits into
RiftCore00:mainfrom
levoski1:issue-7-error-handling-server

Conversation

@levoski1

Copy link
Copy Markdown
Contributor

Closes #7

  • Wraps all ws.send() calls in a safeSend helper that catches and logs send errors instead of throwing
  • Wraps new URL() in try/catch and closes with code 4000 on malformed request URLs
  • Keeps all 50 tests passing

@levibliz
levibliz merged commit 882bcc7 into RiftCore00:main Jun 13, 2026
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants