fix(web): disconnect only the disabled client instead of all sessions#5138
fix(web): disconnect only the disabled client instead of all sessions#5138neatnoise wants to merge 1 commit into
Conversation
817ab69 to
1085d4c
Compare
1085d4c to
d48f044
Compare
Bundle ReportBundle size has no change ✅ |
|
Scanning comments: The std::erase_if complaint — we can't use it because we need to call stop() and join() on each element before erasing. erase_if only takes a predicate, it doesn't support side effects before removal. The rtsp.h struct with 21 fields is a 6-year-old pre-existing issue (we added 1 field: client_cert). |




Description
When disabling a client via the web UI, all active streaming sessions were terminated. This change makes it so only the disabled client's session is disconnected, leaving other clients unaffected.
The client's TLS certificate (already stored in
named_cert_t) is used to identify which session belongs to the disabled client. The cert is propagated from the TLS verify callback throughlaunch_session_tintosession_t, enabling per-client session termination.Issues Fixed or Closed
Roadmap Issues
Type of Change
Checklist
AI Usage