rliszli commented on code in PR #6281:
URL: https://github.com/apache/nifi/pull/6281#discussion_r956980142
##########
minifi/minifi-c2/minifi-c2-service/src/main/java/org/apache/nifi/minifi/c2/service/ConfigService.java:
##########
@@ -229,7 +229,7 @@ public Response heartbeat(
configuration =
configurationProviderValue.getConfiguration();
} catch (ConfigurationProviderException cpe) {
logger.warn("No flow available for agent class " +
agentClass + ", returning No Content (204)");
- response = Response.noContent().build();
+ response = Response.ok(new C2HeartbeatResponse()).build();
Review Comment:
Nice catch, thank you. Did this change because the agents cannot handle the
204 responses with empty body. Updated the warning message accordingly.
--
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]
