GitHub user btwood reopened a pull request:

    https://github.com/apache/nifi/pull/2111

    NIFI-4326 - ExtractEmailHeaders.java unhandled NullPointerException

    Resolve a null pointer exception when there is no recipient available in a 
TO, CC, or BCC header field.
    
    No JIRA, although I did pull and build and test this change. It fixes the 
previously broken case.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/btwood/nifi patch-1

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/nifi/pull/2111.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2111
    
----
commit afec1a35d9c36a4fc35f3a23d7c781c3853ba9a7
Author: btwood <[email protected]>
Date:   2017-08-25T20:37:50Z

    Update ExtractEmailHeaders.java
    
    Resolve a null pointer exception when there is no recipient available in a 
TO, CC, or BCC header field.

commit 47f0fd8cf2fc005b4d71ba4764fde4fa2ec10d4f
Author: btwood <[email protected]>
Date:   2017-08-31T19:10:13Z

    NIFI-4326 Fix strict address parsing in ExtractEmailAttachments.java
    
    This uses getHeader() instead of getFrom() in order to avoid strict 
addressing. 
    This matches changes to ExtractEmailHeaders.java
    
    Postfix and other mail servers are much more linient in accepting 
addresses. Using strict = false from getHeaders() we can accept more addresses 
that otherwise would be rejected.

commit 437e9b627f4976ca7587d21b7c7da97a0dc5f371
Author: btwood <[email protected]>
Date:   2017-08-31T19:11:16Z

    NIFI-4326 Fix NullPointerException and strict addressing
    
    This uses parseHeader() instead of getFrom() and getRecipients() in order 
to avoid strict addressing.
    It also checks for null to solve a null pointer exception.
    By contract, this processor should grab information "if available". Which 
means it should not fail if the info is unavailable.

commit 4380b4deacbf00a324085a55085e33e858318073
Author: btwood <[email protected]>
Date:   2017-09-06T20:24:18Z

    NIFI-4326 Added property for strict addresses 
    
    Added a property and corresponding variable for setting strict addressing. 
The default is ON so as not to impact current users relying on the processor to 
reject weakly addressed email.

commit ccfb2ee3c2725801978d9de0402a2e4656686920
Author: Kevin Doran <[email protected]>
Date:   2017-09-11T17:24:47Z

    NIFI-4326: Add unit test cases

commit 2ad69672602f76fb09a083c96adf74dc3c868240
Author: btwood <[email protected]>
Date:   2017-09-13T13:53:42Z

    Merge pull request #1 from kevdoran/NIFI-4326-extract-email-headers
    
    NIFI-4326: Add unit test cases, clean up code, correct processor 
documentation

commit 8308cabb9cb5a2a9984d03817d3882e85a046fe2
Author: btwood <[email protected]>
Date:   2017-09-13T21:53:40Z

    NIFI-4326 Add non-strict email address parsing
    
    Added mailSession properties for non-strict address parsing to mirror what 
was done in the ExtractEmailHeaders.java
    
    This relaxes the requirement for valid email addresses. Email addresses are 
not used in this processor and has no impact on extracting attachments. In 
order to process more emails and extract more attachments, the 
mail.mime.address.strict property was set to false. This was done with a 
variable in order to make it easier to add a property to the processor later if 
desired.

----


---

Reply via email to