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

    https://github.com/apache/nifi/pull/2544#discussion_r175079382
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java
 ---
    @@ -520,6 +521,27 @@ public void onTrigger(final ProcessContext context, 
final ProcessSession session
                         new Object[]{request.getRemoteAddr(), e});
                 session.remove(flowFile);
                 return;
    +        } catch (final FlowFileAccessException e) {
    +            // some bad requests can produce a IOException on the HTTP 
stream, which makes a FlowFileAccessException to
    +            // be thrown. We should handle these cases here, while other 
FlowFileAccessException are re-thrown
    +            if (!(e.getCause() != null && e.getCause() instanceof 
FlowFileAccessException
    --- End diff --
    
    yes, thank you very much for your comment and your help @markap14 . I am 
updating the PR accordingly. Thanks.


---

Reply via email to