Skip to content

Commit

Permalink
[process] Fix a very nasty bug in processing messages
Browse files Browse the repository at this point in the history
In some cases, we could have forgotten to parse the incoming message. We would have 2x messages in the TCP stream, we would consume the first one but forgot about the second one
  • Loading branch information
mcopik committed Oct 17, 2024
1 parent 1b88f06 commit 1484e3c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion process/controller/src/remote.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ namespace praas::process::remote {
// Three types of messages require long payloads:
// invoke
// put
std::visit(
consumed = std::visit(
common::message::overloaded{
[this, buffer, conn = conn.get()](common::message::InvocationRequestPtr& invoc
) mutable -> bool { return _handle_invocation(*conn, invoc, buffer); },
Expand Down

0 comments on commit 1484e3c

Please sign in to comment.