Skip to content

Conversation

@pedroanastacio
Copy link
Member

@pedroanastacio pedroanastacio commented Jan 8, 2026

Description

  • The socket server is restarting too frequently in production. This adjustment is an attempted solution, as an unhandled request was found that would crash the socket server if the API returned an error.

Summary

  • Adds error handler to connection state request
  • Adds retry logic to connection state request if API return an 5xx error.

Checklist

  • I reviewed my PR code before submitting
  • I ensured that the implementation is working correctly and did not impact other parts of the app
  • I mentioned the PR link in the task

Copy link
Member

@guimroque guimroque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 Production Promotion Checklist

This PR is being merged to main. Please verify the items below:

🔒 Security

  • No secrets/credentials exposed in code
  • No staging/development URLs hardcoded
  • No debug flags enabled

✅ Quality

  • Build passed on staging
  • Tests passed on staging
  • Code was reviewed on staging

🔄 Integrity

✅ Branch main is synchronized

⚠️ Issues Found

🟡 Warnings (manual review)

  • Console statement: console.log( em packages/socket-server/src/socket/index.ts:88

Copy link
Member

@guimroque guimroque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Validation passed. PR approved for merge to main.

@pedroanastacio pedroanastacio merged commit 61e3cfe into main Jan 8, 2026
1 check failed
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.

3 participants