[GitHub] [nifi] ChrisSamo632 commented on a change in pull request #5068: NIFI-8516 Enabled HTTPS and Single User Authentication by default

2021-05-17 Thread GitBox
ChrisSamo632 commented on a change in pull request #5068: URL: https://github.com/apache/nifi/pull/5068#discussion_r634095398 ## File path: nifi-docker/dockermaven/sh/start.sh ## @@ -33,15 +33,29 @@ if [ ! -z "${NIFI_JVM_DEBUGGER}" ]; then fi # Establish baseline properties

[GitHub] [nifi] ChrisSamo632 commented on a change in pull request #5068: NIFI-8516 Enabled HTTPS and Single User Authentication by default

2021-05-17 Thread GitBox
ChrisSamo632 commented on a change in pull request #5068: URL: https://github.com/apache/nifi/pull/5068#discussion_r634093446 ## File path: nifi-docker/dockerhub/sh/start.sh ## @@ -33,15 +33,29 @@ if [ ! -z "${NIFI_JVM_DEBUGGER}" ]; then fi # Establish baseline properties -

[jira] [Commented] (NIFI-8461) Queue reports items but cannot list them

2021-05-17 Thread KevinSky (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17346518#comment-17346518 ] KevinSky commented on NIFI-8461: I found something new. If you set executeScript processor

[GitHub] [nifi] github-actions[bot] closed pull request #2773: NIFI-5155: Add host address info to bulletins

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2773: URL: https://github.com/apache/nifi/pull/2773 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #2686: NIFI-5166 - Deep learning classification and regression processor wit…

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2686: URL: https://github.com/apache/nifi/pull/2686 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #2715: Nifi-5171 - fixed Yandex Jersey issues by adding dependency to POM and modified A…

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2715: URL: https://github.com/apache/nifi/pull/2715 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #2507: Nifi 3093

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2507: URL: https://github.com/apache/nifi/pull/2507 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #2521: NIFI-4946 nifi-spark-bundle : Adding support for pyfiles, file, jars options

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2521: URL: https://github.com/apache/nifi/pull/2521 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #2629: NIFI-5070 Fix java.sql.SQLException: ERROR 1101 (XCL01): ResultSet is…

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2629: URL: https://github.com/apache/nifi/pull/2629 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #2866: NIFI-4710 Kerberos support for user auth in Docker instances

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #2866: URL: https://github.com/apache/nifi/pull/2866 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #3228: NIFI-5897 Update GetFile and ListFile Min Age to handle future dated …

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #3228: URL: https://github.com/apache/nifi/pull/3228 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #3194: NIFI-5860 - Add GCPReportingTask

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #3194: URL: https://github.com/apache/nifi/pull/3194 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #3311: NIFI-6030 Added MongoDBStringLookupService.

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #3311: URL: https://github.com/apache/nifi/pull/3311 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #3051: NIFI-5642: QueryCassandra processor : output FlowFiles as soon fetch_size is reached

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #3051: URL: https://github.com/apache/nifi/pull/3051 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #3343: NIFI-5572

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #3343: URL: https://github.com/apache/nifi/pull/3343 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[GitHub] [nifi] github-actions[bot] closed pull request #3286: NIFI-5988 Fixed validation issues on database and collection names.

2021-05-17 Thread GitBox
github-actions[bot] closed pull request #3286: URL: https://github.com/apache/nifi/pull/3286 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servic

[jira] [Updated] (NIFI-8609) Improve efficiency of converting Record object to Avro GenericRecord object

2021-05-17 Thread Mark Payne (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8609?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Payne updated NIFI-8609: - Fix Version/s: 1.14.0 Status: Patch Available (was: Open) > Improve efficiency of converting R

[GitHub] [nifi] markap14 opened a new pull request #5080: NIFI-8609: Added unit test that is ignored so that it can be manually…

2021-05-17 Thread GitBox
markap14 opened a new pull request #5080: URL: https://github.com/apache/nifi/pull/5080 … run for testing performance before/after changes to AvroTypeUtil. Updated AvroTypeUtil to be more efficient by not using Record.getValue() and instead iterating over the Map of values directly. getVal

[jira] [Created] (NIFI-8609) Improve efficiency of converting Record object to Avro GenericRecord object

2021-05-17 Thread Mark Payne (Jira)
Mark Payne created NIFI-8609: Summary: Improve efficiency of converting Record object to Avro GenericRecord object Key: NIFI-8609 URL: https://issues.apache.org/jira/browse/NIFI-8609 Project: Apache NiFi

[GitHub] [nifi] exceptionfactory commented on pull request #5068: NIFI-8516 Enabled HTTPS and Single User Authentication by default

2021-05-17 Thread GitBox
exceptionfactory commented on pull request #5068: URL: https://github.com/apache/nifi/pull/5068#issuecomment-842639203 @ChrisSamo632, I updated the Docker startup scripts to enable HTTPS on port 8443 as the default configuration. @thenatog In light of the Docker images previously ref

[GitHub] [nifi] thenatog commented on pull request #5068: NIFI-8516 Enabled HTTPS and Single User Authentication by default

2021-05-17 Thread GitBox
thenatog commented on pull request #5068: URL: https://github.com/apache/nifi/pull/5068#issuecomment-842626077 Hmm, yes, it does appear to be a bit of a mix between 8443 and 9443. Perhaps we should continue with 9443 as that's what officially existed in docs. -- This is an automated mess

[GitHub] [nifi] exceptionfactory commented on pull request #5068: NIFI-8516 Enabled HTTPS and Single User Authentication by default

2021-05-17 Thread GitBox
exceptionfactory commented on pull request #5068: URL: https://github.com/apache/nifi/pull/5068#issuecomment-842620761 > Tested this out and seems to work well. One question would be, why specifically is the default port 9443? Typically, most documentation refers to NiFi using 8443 for sec

[GitHub] [nifi] thenatog commented on pull request #5025: NIFI-8466: Resolving offload bug with Single Node load balanced queues

2021-05-17 Thread GitBox
thenatog commented on pull request #5025: URL: https://github.com/apache/nifi/pull/5025#issuecomment-842614242 I will test this out with a 3 node cluster -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [nifi] thenatog commented on pull request #5068: NIFI-8516 Enabled HTTPS and Single User Authentication by default

2021-05-17 Thread GitBox
thenatog commented on pull request #5068: URL: https://github.com/apache/nifi/pull/5068#issuecomment-842611731 Tested this out and seems to work well. One question would be, why specifically is the default port 9443? Typically, most documentation refers to NiFi using 8443 for secure NiFi:

[GitHub] [nifi] jfrazee commented on pull request #5078: NIFI-8501: Setup Azure blob client side encryption

2021-05-17 Thread GitBox
jfrazee commented on pull request #5078: URL: https://github.com/apache/nifi/pull/5078#issuecomment-842542939 Reviewing -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For

[GitHub] [nifi] scottyaslan commented on a change in pull request #5060: NIFI-8520 - Parameter Contexts - Show the wrong information of referencing compon…

2021-05-17 Thread GitBox
scottyaslan commented on a change in pull request #5060: URL: https://github.com/apache/nifi/pull/5060#discussion_r633759203 ## File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-parameter-contexts.js ## @@ -23

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1069: MINIFICPP-1084: Linter check should be platform independent

2021-05-17 Thread GitBox
lordgamez commented on a change in pull request #1069: URL: https://github.com/apache/nifi-minifi-cpp/pull/1069#discussion_r633276351 ## File path: libminifi/test/KamikazeProcessor.h ## @@ -33,14 +35,14 @@ namespace minifi { namespace processors { // GenerateFlowFile Class c

[GitHub] [nifi] phrocker commented on pull request #5077: NIFI-8604 Upgraded Apache Accumulo to 2.0.1

2021-05-17 Thread GitBox
phrocker commented on pull request #5077: URL: https://github.com/apache/nifi/pull/5077#issuecomment-842350778 @thenatog and @exceptionfactory I haven't tested this bundle with the updated pom specifically but have tested 2.0.1 -> 2.0.0 and there should be no issues previously. The only p

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #1069: MINIFICPP-1084: Linter check should be platform independent

2021-05-17 Thread GitBox
fgerlits commented on a change in pull request #1069: URL: https://github.com/apache/nifi-minifi-cpp/pull/1069#discussion_r633631260 ## File path: CONTRIB.md ## @@ -33,8 +33,15 @@ Apache NiFi MiNiFi C++ is a review then commit community. As a result, we will r review. We ask

[GitHub] [nifi-minifi-cpp] martinzink commented on a change in pull request #1069: MINIFICPP-1084: Linter check should be platform independent

2021-05-17 Thread GitBox
martinzink commented on a change in pull request #1069: URL: https://github.com/apache/nifi-minifi-cpp/pull/1069#discussion_r633506656 ## File path: libminifi/test/KamikazeProcessor.h ## @@ -33,14 +35,14 @@ namespace minifi { namespace processors { // GenerateFlowFile Class

[GitHub] [nifi] exceptionfactory commented on pull request #5077: NIFI-8604 Upgraded Apache Accumulo to 2.0.1

2021-05-17 Thread GitBox
exceptionfactory commented on pull request #5077: URL: https://github.com/apache/nifi/pull/5077#issuecomment-842372044 Thanks for the feedback @phrocker! Given the apparent low impact of the changes from 2.0.0 from 2.0.1, I'm comfortable moving forward, however, I defer to you and @thenat

[GitHub] [nifi] mcgilman commented on a change in pull request #5072: WIP NIFI-8490: Adding inherited parameter contexts

2021-05-17 Thread GitBox
mcgilman commented on a change in pull request #5072: URL: https://github.com/apache/nifi/pull/5072#discussion_r633642994 ## File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ParameterContextResource.java #

[GitHub] [nifi] scottyaslan commented on a change in pull request #5060: NIFI-8520 - Parameter Contexts - Show the wrong information of referencing compon…

2021-05-17 Thread GitBox
scottyaslan commented on a change in pull request #5060: URL: https://github.com/apache/nifi/pull/5060#discussion_r633713204 ## File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-parameter-contexts.js ## @@ -23

[GitHub] [nifi-minifi-cpp] arpadboda closed pull request #1066: MINIFICPP-1532: Create a processor to capture resource consumption me…

2021-05-17 Thread GitBox
arpadboda closed pull request #1066: URL: https://github.com/apache/nifi-minifi-cpp/pull/1066 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servi

[GitHub] [nifi] markobean commented on pull request #5061: NIFI-8523 Update secure ftp processors to make configurable which algorithms, ciphers and message authentication codes are allowed to be used

2021-05-17 Thread GitBox
markobean commented on pull request #5061: URL: https://github.com/apache/nifi/pull/5061#issuecomment-841874983 Thanks for the updates, especially the Variable Registry. LGTM -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #1057: MINIFICPP-1537 - Log heartbeats on demand

2021-05-17 Thread GitBox
fgerlits commented on a change in pull request #1057: URL: https://github.com/apache/nifi-minifi-cpp/pull/1057#discussion_r633553516 ## File path: libminifi/include/c2/HeartbeatJsonSerializer.h ## @@ -0,0 +1,59 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [nifi] scottyaslan commented on a change in pull request #5060: NIFI-8520 - Parameter Contexts - Show the wrong information of referencing compon…

2021-05-17 Thread GitBox
scottyaslan commented on a change in pull request #5060: URL: https://github.com/apache/nifi/pull/5060#discussion_r633713204 ## File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-parameter-contexts.js ## @@ -23

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #1069: MINIFICPP-1084: Linter check should be platform independent

2021-05-17 Thread GitBox
fgerlits commented on a change in pull request #1069: URL: https://github.com/apache/nifi-minifi-cpp/pull/1069#discussion_r633631260 ## File path: CONTRIB.md ## @@ -33,8 +33,15 @@ Apache NiFi MiNiFi C++ is a review then commit community. As a result, we will r review. We ask

[GitHub] [nifi] mcgilman commented on a change in pull request #5072: WIP NIFI-8490: Adding inherited parameter contexts

2021-05-17 Thread GitBox
mcgilman commented on a change in pull request #5072: URL: https://github.com/apache/nifi/pull/5072#discussion_r633642994 ## File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ParameterContextResource.java #

[GitHub] [nifi] thenatog opened a new pull request #5079: NIFI-8025 - Refactored AccessResource SAML and OIDC endpoints into separate classes

2021-05-17 Thread GitBox
thenatog opened a new pull request #5079: URL: https://github.com/apache/nifi/pull/5079 NIFI-8025 - Moved SAML to separate class and tested to be working. NIFI-8025 - Added license to new files. NIFI-8025 - Fixed a SAML logout consumer path. Thank you for submitting

[GitHub] [nifi-minifi-cpp] fgerlits commented on a change in pull request #1057: MINIFICPP-1537 - Log heartbeats on demand

2021-05-17 Thread GitBox
fgerlits commented on a change in pull request #1057: URL: https://github.com/apache/nifi-minifi-cpp/pull/1057#discussion_r633553516 ## File path: libminifi/include/c2/HeartbeatJsonSerializer.h ## @@ -0,0 +1,59 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [nifi] exceptionfactory commented on pull request #5077: NIFI-8604 Upgraded Apache Accumulo to 2.0.1

2021-05-17 Thread GitBox
exceptionfactory commented on pull request #5077: URL: https://github.com/apache/nifi/pull/5077#issuecomment-842372044 Thanks for the feedback @phrocker! Given the apparent low impact of the changes from 2.0.0 from 2.0.1, I'm comfortable moving forward, however, I defer to you and @thenat

[GitHub] [nifi] phrocker commented on pull request #5077: NIFI-8604 Upgraded Apache Accumulo to 2.0.1

2021-05-17 Thread GitBox
phrocker commented on pull request #5077: URL: https://github.com/apache/nifi/pull/5077#issuecomment-842350778 @thenatog and @exceptionfactory I haven't tested this bundle with the updated pom specifically but have tested 2.0.1 -> 2.0.0 and there should be no issues previously. The only p

[GitHub] [nifi-minifi-cpp] martinzink commented on a change in pull request #1069: MINIFICPP-1084: Linter check should be platform independent

2021-05-17 Thread GitBox
martinzink commented on a change in pull request #1069: URL: https://github.com/apache/nifi-minifi-cpp/pull/1069#discussion_r633506656 ## File path: libminifi/test/KamikazeProcessor.h ## @@ -33,14 +35,14 @@ namespace minifi { namespace processors { // GenerateFlowFile Class

[GitHub] [nifi-minifi-cpp] arpadboda closed pull request #1066: MINIFICPP-1532: Create a processor to capture resource consumption me…

2021-05-17 Thread GitBox
arpadboda closed pull request #1066: URL: https://github.com/apache/nifi-minifi-cpp/pull/1066 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this servi

[jira] [Updated] (NIFI-8608) PutS3Object processor missing SHA256 Hash on https endpoint. Enhancement

2021-05-17 Thread Denis Jakupovic (Jira)
-b08 java/1.8.0_282 vendor/Oracle_Corporation Content-Length: 25600 Amz-Sdk-Invocation-Id: 8a531ba6-7aa2-e679-0c5b-84779636e71e Amz-Sdk-Request: attempt=1;max=1 Amz-Sdk-Retry: 0/0/500 Authorization: AWS4-HMAC-SHA256 Credential=/20210517/us-east-1/s3/aws4_request

[jira] [Created] (NIFI-8608) PutS3Object processor missing SHA256 Hash on https endpoint. Enhancement

2021-05-17 Thread Denis Jakupovic (Jira)
: 25600 Amz-Sdk-Invocation-Id: 8a531ba6-7aa2-e679-0c5b-84779636e71e Amz-Sdk-Request: attempt=1;max=1 Amz-Sdk-Retry: 0/0/500 Authorization: AWS4-HMAC-SHA256 Credential=C6anqCx1lMuGd7wAwUPO/20210517/us-east-1/s3/aws4_request, SignedHeaders=amz-sdk-invocation-id;amz-sdk-request;amz-sdk-retry

[GitHub] [nifi-minifi-cpp] lordgamez commented on a change in pull request #1069: MINIFICPP-1084: Linter check should be platform independent

2021-05-17 Thread GitBox
lordgamez commented on a change in pull request #1069: URL: https://github.com/apache/nifi-minifi-cpp/pull/1069#discussion_r633276351 ## File path: libminifi/test/KamikazeProcessor.h ## @@ -33,14 +35,14 @@ namespace minifi { namespace processors { // GenerateFlowFile Class c

[jira] [Commented] (NIFI-8333) Issues with invokehttp/SSL Service after "Terminate Task" (e.g. due to long running transactions) and misleading error-message

2021-05-17 Thread Andreas Klingenstein (Jira)
[ https://issues.apache.org/jira/browse/NIFI-8333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17345943#comment-17345943 ] Andreas Klingenstein commented on NIFI-8333: We don't have any custom processo