cunningt commented on code in PR #1355:
URL:
https://github.com/apache/camel-spring-boot/pull/1355#discussion_r1934742473
##########
components-starter/camel-platform-http-starter/src/main/java/org/apache/camel/component/platform/http/springboot/SpringBootPlatformHttpBinding.java:
##########
@@ -168,6 +168,7 @@ protected void doWriteDirectResponse(Message message,
HttpServletResponse respon
} finally {
IOHelper.close(new Closeable[]{is, os});
}
+ stream.close();
Review Comment:
@davsclaus Really sorry, but I'm still confused here about why the stream
variable can't be closed out at the end of the if block because it looks like
it goes out of scope there - is there somewhere we can close it out so we don't
have to deal with this Coverity test failing?
--
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]