exceptionfactory commented on pull request #4892: URL: https://github.com/apache/nifi/pull/4892#issuecomment-797705226
> Left a few comments after very quickly going through it. I'd definitely like to have a few people looking into these changes as this processor is one of the most widely used. Thanks for the detailed feedback and questions @pvillard31. I am open to reverting some of the changes to `InvokeHTTP`, but hopefully the answers on particular changes explain why it seemed to make sense to include the adjustments. The new `InvokeHTTPTest` attempts to cover a wide array of use cases, which also helped expose some of the limited use and dead code lines in `InvokeHTTP`. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
