lordgamez commented on code in PR #1494:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1494#discussion_r1101511272
##########
extensions/http-curl/protocols/RESTSender.cpp:
##########
@@ -162,6 +162,7 @@ C2Payload RESTSender::sendPayload(const std::string& url,
const Direction direct
if (payload.getOperation() == Operation::TRANSFER) {
auto read =
std::make_unique<utils::HTTPReadCallback>(std::numeric_limits<size_t>::max());
client.setReadCallback(std::move(read));
+ client.setRequestHeader("Accept",
"application/vnd.minifi-c2+json;version=1, text/yml");
Review Comment:
I think the json format is the default now in the docker tests, only a few
tests are set specifically to use yaml. So I think the nifi C2 tests are
running with the json config now so I suppose that should be okay. Maybe 1 more
test could be added to test both versions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
