[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

[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 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_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_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 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_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_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_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_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 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 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 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_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_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_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_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-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_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_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_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 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't

[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 ap

[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 @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 '^]'. 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 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 localhost

[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 #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 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 issue: Note how POIs LICENSE makes mention to CurvesAPI https://github.com/apache/poi/blob/trunk/legal/LICENSE#L459

[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 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 source

[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 #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 #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_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_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_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_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_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_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 pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70721418 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70574245 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70714764 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70714800 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #475: NIFI-2026 - Add Maven profile to compile nifi-hadoop...

2016-07-21 Thread trixpan
Github user trixpan closed the pull request at: https://github.com/apache/nifi/pull/475 --- 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

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

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

[GitHub] nifi issue #483: NIFI-1899 - Introduce ExtractEmailAttachments and ExtractEm...

2016-07-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/483 @JPercivall Beautiful code! Loved it. Interesting to see your programming thinking. Great lesson (specially because I tried to achieve a similar - though more covoluted - logic previously and failed

[GitHub] nifi issue #483: NIFI-1899 - Introduce ExtractEmailAttachments and ExtractEm...

2016-07-16 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/483 @JPercivall Fixing the rebase. Please hold --- 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 #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70612760 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70612776 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-13 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70615355 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70366557 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-08 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70157262 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/pom.xml --- @@ -0,0 +1,75 @@ + + +http://maven.apache.org/POM/4.0.0; xmlns:xsi

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-08 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70157601 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java --- @@ -0,0

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-08 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70157551 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java --- @@ -0,0

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-08 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70157622 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailHeaders.java --- @@ -0,0 +1,233

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70180440 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -0,0 +1,385

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70180458 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70180468 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -0,0 +1,385

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70180484 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java --- @@ -0,0

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70180515 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java --- @@ -0,0

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70180543 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java --- @@ -0,0

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70181608 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java --- @@ -0,0 +1,385

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-10 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70196911 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi issue #483: NIFI-1899 - Introduce ExtractEmailAttachments and ExtractEm...

2016-07-10 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/483 @ijokarumawak Still not clear to me how you are reaching the error. Also, note the QUIT message is not a requirement, a client may chose to send more than a message per session, example

[GitHub] nifi issue #483: NIFI-1899 - Introduce ExtractEmailAttachments and ExtractEm...

2016-07-11 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/483 @ijokarumawak - thanks for looking at it. So just for me to confirm: PutEmail connects and tries to use SSL. This fails. It holds the socket PutEmail connects again

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70287734 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70288235 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/handler/SMTPMessageHandlerFactory.java

[GitHub] nifi pull request #483: NIFI-1899 - Introduce ExtractEmailAttachments and Ex...

2016-07-11 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70286727 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java --- @@ -0,0

[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 #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 #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 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 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 #710: NIFI-1148 added IMAP/POP3 support

2016-07-23 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r71973547 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java --- @@ -0,0

[GitHub] nifi pull request #712: NIFI-2374 and NIFI-2375 - Minor improve to documenta...

2016-07-23 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/712 NIFI-2374 and NIFI-2375 - Minor improve to documentation and version bump NIFI-2374 - Today when I was about to raise an ISSUE I've noticed that although the IdentifyMimeType documentation provides

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

2016-07-23 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r71973345 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java --- @@ -0,0

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

2016-07-23 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r71973519 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java --- @@ -0,0

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

2016-07-23 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/710#discussion_r71973968 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java --- @@ -0,0

[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

[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 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

[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 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 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 issue #1462: NIFI-819 - Extend GetHTTP to use dynamically produce HTTP ...

2017-02-01 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1462 @apiri A long time ago in a galaxy far far away you raised this JIRA ticket. Does this PR addresses what you had in mind? Cheers --- If your project is set up for it, you can

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically produc...

2017-02-01 Thread trixpan
GitHub user trixpan opened a pull request: https://github.com/apache/nifi/pull/1462 NIFI-819 - Extend GetHTTP to use dynamically produce HTTP headers upo… …n a request Thank you for submitting a contribution to Apache NiFi. In order to streamline the review

[GitHub] nifi issue #1468: NIFI-3430 adds support for date format property to parse i...

2017-02-02 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/1468 Nick, thank you for the contribution. would you mind squashing the commits? --- 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 #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038019 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -62,18 +62,12

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038165 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -102,6 +96,11

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038013 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -62,18 +62,12

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038126 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -102,6 +96,11

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99039524 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -62,18 +62,12

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99039594 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -102,6 +96,11

[GitHub] nifi pull request #1462: NIFI-819 - Extend GetHTTP to use dynamically add HT...

2017-02-01 Thread trixpan
Github user trixpan commented on a diff in the pull request: https://github.com/apache/nifi/pull/1462#discussion_r99038321 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java --- @@ -467,6 +486,18

  1   2   3   4   5   6   7   >