Skip to content

Conversation

@tieneupin
Copy link
Contributor

@tieneupin tieneupin commented Dec 11, 2025

Merge PR #715 first.

Replaced Client-Side Websocket Connection-Based Logging with HTTPX One
The websocket connection between the client and the backend server has been unstable, prone to error, and unable to fulfil its role of sending logs to the backend for subsequent forwarding to the logging dashboard. In this PR, the log forwarding logic has been replaced by a HTTPX-based log handler that sends batches of logs to a backend FastAPI endpoint. It also removes the now-defunct websocket-related components from the client end.

Simplifcation of pyproject.toml Dependency Keys
Consequently, the optional dependency keys needed for our pyproject.toml file can also been streamlined. As of this PR, the client and instrument-server optional dependency keys are no longer needed, as the packages needed to run the instrument server are a subset of those needed to run the backend server, and have thus been integrated into the essential dependencies section. The documentation and GitHub workflows have thus been updated to reflect this.

@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

❌ Patch coverage is 93.79845% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.19%. Comparing base (cdbf693) to head (c9fe70c).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #716      +/-   ##
==========================================
+ Coverage   42.49%   43.19%   +0.70%     
==========================================
  Files          94       94              
  Lines       10347    10239     -108     
  Branches     1346     1332      -14     
==========================================
+ Hits         4397     4423      +26     
+ Misses       5733     5597     -136     
- Partials      217      219       +2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin tieneupin marked this pull request as ready for review December 15, 2025 12:44
@tieneupin tieneupin self-assigned this Dec 15, 2025
@tieneupin tieneupin added the documentation Improvements or additions to documentation label Dec 15, 2025
@tieneupin tieneupin added enhancement New feature or request server Relates to the server component client Relates to the client component dependencies labels Dec 15, 2025
@tieneupin tieneupin mentioned this pull request Dec 18, 2025
@stephen-riggs stephen-riggs merged commit 6e2c3b9 into main Dec 19, 2025
17 checks passed
@stephen-riggs stephen-riggs deleted the replace-client-websocket branch December 19, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Relates to the client component dependencies documentation Improvements or additions to documentation enhancement New feature or request server Relates to the server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants