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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 - 100 of 658 matches
Mail list logo