8000 refactor: Server-client implementation by Quozul · Pull Request #11 · Quozul/PicoLimbo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

refactor: Server-client implementation #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Jun 1, 2025
Merged

refactor: Server-client implementation #11

merged 4 commits into from
Jun 1, 2025

Conversation

Quozul
Copy link
Owner
@Quozul Quozul commented May 30, 2025

Refactor Summary

  • Removed explicit mutex handling from handlers, eliminating the need to manually acquire the client lock within handler code and improving type clarity.
  • Updated handlers to return Result types, enabling robust error handling and centralized error catching.
  • Eliminated usage of anyhow in favor of precise, well-defined error types for clearer error semantics.
  • Enhanced debug and error logging for better traceability and easier troubleshooting.
  • Added detection of client graceful disconnections, allowing the server to broadcast user-friendly departure messages.

@Quozul Quozul merged commit f381cf5 into master Jun 1, 2025
4 checks passed
@Quozul Quozul deleted the refactor/server-client-implementation branch June 1, 2025 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Quozul
0