Github user phrocker commented on a diff in the pull request:

    https://github.com/apache/nifi-minifi-cpp/pull/191#discussion_r151568015
  
    --- Diff: extensions/http-curl/protocols/RESTReceiver.cpp ---
    @@ -65,38 +65,8 @@ void RESTReceiver::initialize(const 
std::shared_ptr<core::controller::Controller
       }
     }
     int16_t RESTReceiver::heartbeat(const C2Payload &payload) {
    -  std::string operation_request_str = getOperation(payload);
    --- End diff --
    
    I assume serializeJsonRootPayload has evolved to nest the payloads? Have 
you tested this with a C2 server? We should do that before merging. 


---

Reply via email to