Skip to content

Use UUID for websocket connection for logging from the instrument server#529

Merged
tieneupin merged 1 commit intomainfrom
attempt-to-sort-out-websocket-connections
Mar 24, 2025
Merged

Use UUID for websocket connection for logging from the instrument server#529
tieneupin merged 1 commit intomainfrom
attempt-to-sort-out-websocket-connections

Conversation

@d-j-hatton
Copy link
Contributor

This needs testing but may resolve the websocket ID clashes we've seen

@d-j-hatton d-j-hatton requested a review from tieneupin March 21, 2025 12:05
@codecov
Copy link

codecov bot commented Mar 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.86%. Comparing base (7e41242) to head (cb42ac9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #529   +/-   ##
=======================================
  Coverage   28.86%   28.86%           
=======================================
  Files          79       79           
  Lines       10290    10290           
  Branches     1364     1364           
=======================================
  Hits         2970     2970           
  Misses       7219     7219           
  Partials      101      101           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tieneupin tieneupin merged commit 324f70b into main Mar 24, 2025
17 checks passed
@tieneupin tieneupin deleted the attempt-to-sort-out-websocket-connections branch March 24, 2025 09:35
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