Merge pull request #29 from /issues/28

vortigont · web-flow · commit 7be1742797f9 · 2025-01-28T22:51:02.000+03:00
fix(sse): break loop after erasing client
diff --git a/src/AsyncEventSource.cpp b/src/AsyncEventSource.cpp
@@ -398,6 +398,7 @@ void AsyncEventSource::_handleDisconnect(AsyncEventSourceClient *client) {
   for (auto i = _clients.begin(); i != _clients.end(); ++i) {
     if (i->get() == client) {
       _clients.erase(i);
+      break;
     }
   }
   _adjust_inflight_window();

Original file line number	Diff line number	Diff line change
`@@ -398,6 +398,7 @@ void AsyncEventSource::_handleDisconnect(AsyncEventSourceClient *client) {`
`398`	`398`	`for (auto i = _clients.begin(); i != _clients.end(); ++i) {`
`399`	`399`	`if (i->get() == client) {`
`400`	`400`	`_clients.erase(i);`
	`401`	`+ break;`
`401`	`402`	`}`
`402`	`403`	`}`
`403`	`404`	`_adjust_inflight_window();`