[GitHub] nifi pull request #964: Port NIFI-2688 to NIFI 1.0.0 RC1 branch

2016-08-28 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/964 Port NIFI-2688 to NIFI 1.0.0 RC1 branch adjust ConfigEncryptionToolTest datePattern regex to cater for timezones with 4 characters You can merge this pull request into a Git repository by running

[GitHub] nifi pull request #963: NIFI-2688 - adjust ConfigEncryptionToolTest datePatt...

2016-08-28 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/963 NIFI-2688 - adjust ConfigEncryptionToolTest datePattern regex to cate… regex to cater for timezones with 4 characters You can merge this pull request into a Git repository by running: $ git

[GitHub] nifi pull request #961: NIFI-2685 - Configure surefire to use JAVA Headless ...

2016-08-28 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/961 NIFI-2685 - Configure surefire to use JAVA Headless mode You can merge this pull request into a Git repository by running: $ git pull https://github.com/trixpan/nifi NIFI-2685-master

[GitHub] nifi pull request #960: NIFI-2685 - Configure maven to use java headless mod...

2016-08-28 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/960 NIFI-2685 - Configure maven to use java headless mode during build You can merge this pull request into a Git repository by running: $ git pull https://github.com/trixpan/nifi NIFI-2685

[GitHub] nifi issue #785: NIFI-2341 - Introduce ParseCEF processor

2016-08-27 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/785 Ready for review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] nifi issue #785: NIFI-2341 - Introduce ParseCEF processor

2016-08-27 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/785 figured out what it is. I needed to exclude slf4j-log4j12 from parCEFone dependency. In the process found a few bugs. Fixing. :-) --- If your project is set up for it, you can reply to this

[GitHub] nifi issue #785: NIFI-2341 - Introduce ParseCEF processor

2016-08-27 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/785 @mattyb149 - For some reason I don't truly grasp TestGetJMSQueue start to fail once this processor is introduced to standard-processors Other than that code looks to be working as exp

[GitHub] nifi issue #941: NIFI-2662 - Fix style violations previously missed

2016-08-25 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/941 closed as per @mcgilman request --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] nifi pull request #941: NIFI-2662 - Fix style violations previously missed

2016-08-25 Thread trixpan
Github user trixpan closed the pull request at: https://github.com/apache/nifi/pull/941 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] nifi issue #941: NIFI-2662 - Fix style violations previously missed

2016-08-25 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/941 @JPercivall - no worries at all. Please note PR940 is already live so we may be worth checking that as well --- If your project is set up for it, you can reply to this email and have

[GitHub] nifi issue #941: NIFI-2662 - Fix style violations previously missed

2016-08-25 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/941 @mattyb149 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] nifi pull request #941: NIFI-2662 - Fix style violations previously missed

2016-08-25 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/941 NIFI-2662 - Fix style violations previously missed Fix styles missed during PR940 You can merge this pull request into a Git repository by running: $ git pull https://github.com/trixpan/nifi

[GitHub] nifi issue #940: NIFI-2662 - Fix checkstyle violations introduced by NIFI-26...

2016-08-25 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/940 previous title and commit message referred to incorrect JIRA number. Correct number according to Github's blame view is NIFI-2635 --- If your project is set up for it, you can reply to

[GitHub] nifi pull request #940: NIFI-2662 - Fix checkstyle violations introduced by ...

2016-08-25 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/940 NIFI-2662 - Fix checkstyle violations introduced by NIFI-2605 You can merge this pull request into a Git repository by running: $ git pull https://github.com/trixpan/nifi NIFI-2662

[GitHub] nifi issue #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-25 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/858 @pvillard31 - reworded validators, unified the regex / split parses to use numeric capture groups/columns --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-25 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r76225271 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java --- @@ -0,0 +1,334

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-25 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r76225290 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java --- @@ -0,0 +1,334

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-25 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r76220146 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java --- @@ -0,0 +1,334

[GitHub] nifi issue #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/858 As you can see the KEY group is documented within NiFi code but the UI lacks such explanation https://github.com/apache/nifi/pull/858/files#diff-d13cbfd36645869513f9d09b76636240R183 --- If

[GitHub] nifi issue #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/858 @pvillard31 The error is expected as the regex does not contain a capture group named KEY. It is a bug as I should detect KEY or whatever value was chosen. But your feedback

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r75580308 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/test/java/org/apache/nifi/processors/enrich/TestQueryWhois.java --- @@ -0,0 +1,187

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r75580295 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java --- @@ -0,0 +1,334

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r75580268 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java --- @@ -0,0 +1,334

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r75579997 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/pom.xml --- @@ -63,5 +63,31 @@ nifi-mock test

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-20 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r75579988 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/pom.xml --- @@ -63,5 +63,31 @@ nifi-mock test

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-17 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/858#discussion_r75210911 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java --- @@ -0,0 +1,334

[GitHub] nifi pull request #875: Nifi 2477

2016-08-16 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/875#discussion_r75027919 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -169,6 +169,81 @@ Now that the User Interface has been secured, we can easily secure

[GitHub] nifi issue #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/858 @pvillard31 given your experience reviewing QueryDNS, would you mind having a look on this one? Different interface, similar outcome. Cheers --- If your project is set up for it, you can

[GitHub] nifi issue #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/858 A gist to help people going https://gist.github.com/trixpan/9c663866d2b02322650bfddde3eefb6e --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] nifi pull request #871: NIFI-2519 - ListenSMTP addresses mismatch between sm...

2016-08-15 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/871 NIFI-2519 - ListenSMTP addresses mismatch between smtp.src attribute … …documentation and attribute.put You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] nifi issue #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-15 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/858 # Note to reviewers: # This is a logging pipeline focused processor that should be capable of reaching to a Whois server and grabbing information that can be used to enrich data. The

[GitHub] nifi pull request #858: NIFI-1971 - Introduce QueryWhois processor

2016-08-15 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/858 NIFI-1971 - Introduce QueryWhois processor supporting batching (i.e. netcat protocol) You can merge this pull request into a Git repository by running: $ git pull https://github.com/trixpan

[GitHub] nifi pull request #856: NIFI-2519 aligned processor to leverage the subethas...

2016-08-14 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/856#discussion_r74700141 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/SmtpConsumer.java --- @@ -0,0 +1,161

[GitHub] nifi issue #856: NIFI-2519 aligned processor to leverage the subethasmtp thr...

2016-08-14 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/856 :+1: Tested manually and also forced some exceptions. ListenSMTP seems to handle errors very well, example: Place breakpoint on `processSession.getProvenanceReporter

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-12 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74527190 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +90,17

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @joewitt I will need some assistance with the L&N issue: Note how POIs LICENSE makes mention to CurvesAPI https://github.com/apache/poi/blob/trunk/legal/LICENSE#

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74527655 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +90,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74527077 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +90,17

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 @olegz thanks for the clarification. Just so that we are on the same page, I am not concerned about the DFM in this case. What I mean by notification is the following: SMTP

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 @olegz have you pushed the notification recently? I got just an abrupt termination ``` $ telnet 0 2525 Trying 0.0.0.0... Connected to 0. Escape character is '^]'

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 Stopping the processor with an agent connected also seems to throw an exception (not sure if expected) ``` 2016-08-12 01:42:06,593 INFO [StandardProcessScheduler Thread-7

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 1. Start ListenSMTP port 2525 with 1 maximum connection, 10 seconds of timeout ``` $ telnet 0 2525 Trying 0.0.0.0... Connected to 0. Escape character is '^]'. 220

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 @olegz is this expected? ``` 2016-08-12 01:23:59,420 ERROR [Timer-Driven Process Thread-6] o.a.nifi.processors.email.ListenSMTP org.apache.nifi.processor.exception.ProcessException

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 @joewitt, @olegz code is a great improvement to the previous code. I do think the the removal of the maximum message size and timeout checks should be addressed as soon as possible: I

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74449761 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -31,101 +30,75

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @JPercivall - Given your review of ExtractEmailAttachments do you mind reviewing this PR? Cheers --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi issue #817: NIFI-2380 - Introduce ExtractTNEFAttachments

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/817 @joewitt - 3rd party test files have been replaced with new files containing only NiFi & MiNiFi logos --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 @olegz You are clearly confusing envelope (the data exchanged within an SMTP session) with header information (the data added to the body of the message after the DATA command). They don&#

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74423248 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -135,29 +89,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74427203 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74425136 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74421721 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74420800 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -135,29 +89,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74418332 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74416389 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -135,29 +89,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74414544 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -166,317 +108,158

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74414087 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -13,89 +13,52

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74364045 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/test/java/org/apache/nifi/processors/email/TestListenSMTP.java --- @@ -13,307 +13,174

[GitHub] nifi issue #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/827 @olegz - Tested the processor and other than the view that attributes should not be removed (as they are useful for many auditing and downstream applications) and a few remarks on SubEthaSMTP

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74363458 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -166,317 +108,158

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74363178 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74362949 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -13,89 +13,52

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74362705 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -166,317 +108,158

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74362680 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74362612 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -13,89 +13,52

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74362397 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -13,89 +13,52

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74362012 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -166,317 +108,158

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74361899 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -166,317 +108,158

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74361570 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -166,317 +108,158

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74361191 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -106,26 +69,17

[GitHub] nifi pull request #827: NIFI-2519 Fixed and refactored ListenSMTP processor

2016-08-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/827#discussion_r74359964 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -13,89 +13,52

[GitHub] nifi pull request #819: NIFI-1965 - Convert DNS_TIMEOUT property to TIME_PER...

2016-08-09 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/819 NIFI-1965 - Convert DNS_TIMEOUT property to TIME_PERIOD @pvillard31 - for some reason this commit ended up on another branch. Mind having a look? my apologies. You can merge this pull

[GitHub] nifi pull request #817: NIFI-2380 - Introduce ExtractTNEFAttachments

2016-08-09 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/817 NIFI-2380 - Introduce ExtractTNEFAttachments a processor to extract attachments embedded within a Transport Neutral Encapsulation Format (TNEF) attachment (i.e. winmail.dat) You can merge this pull

[GitHub] nifi issue #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/496 @pvillard31 addressed both the trailing whitespace (sigh...) and the name fiasco. Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi issue #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-07 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/496 @pvillard31 let me know if you have any other comments as I am waiting for this to be merged to submit a PR covering NIFI-1971 as well Cheers --- If your project is set up for it, you can

[GitHub] nifi pull request #785: NIFI-2341 - Introduce ParseCEF processor

2016-08-04 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/785 NIFI-2341 - Introduce ParseCEF processor * Implements a processor to parse HPE's highly popular Common Event Format * Please note this code relies on a Parser written outside the NiFi s

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-02 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r73103917 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi issue #712: NIFI-2374 and NIFI-2375 - Minor improve to documentation an...

2016-08-01 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/712 @olegz mind having a look on this? Simple PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r73066535 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r73066145 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r73065856 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-08-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r73065633 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-08-01 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/710 @olegz, seems like an option. How about adding an additional help file withbthe basics of what you already found? --- If your project is set up for it, you can reply to this email and have your

[GitHub] nifi issue #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-29 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/496 @pvillard31 Hopefully issues have been addressed now. Cheers --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-29 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72882149 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-29 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72882132 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,272

[GitHub] nifi issue #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/496 @pvillard31 rebase complete and hopefully all issues were addressed. Let me know if I missed something --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72597016 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,269

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72596817 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,269

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72596906 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,269

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72596717 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,269

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72596621 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryDNS.java --- @@ -0,0 +1,269

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72595518 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/test/java/org/apache/nifi/processors/enrich/TestQueryDNS.java --- @@ -0,0 +1,228

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72593682 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java --- @@ -0,0

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72592665 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java --- @@ -0,0

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72592679 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java --- @@ -0,0

[GitHub] nifi pull request #496: NIFI-1965 - Implement QueryDNS Processor

2016-07-28 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/496#discussion_r72592085 --- Diff: nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java --- @@ -0,0

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-07-24 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/710 @olegz what about using the same strategy as PutEmail where the user can configure the socket on runtime? https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-07-24 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/710 @olegz agreed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] nifi issue #710: NIFI-1148 added IMAP/POP3 support

2016-07-24 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/710 Getting errors when trying to use mailtrap.io for POP3 testing: ``` 2016-07-25 10:37:27,706 ERROR [Timer-Driven Process Thread-6] o.a.nifi.processors.email.ConsumePOP3

<    2   3   4   5   6   7   8   >