[jira] [Updated] (MINIFICPP-2346) Speed up MiNiFi Build with Conan package manager (C++)

2024-05-14 Thread James Guzman (Medel) (Jira)


 [ 
https://issues.apache.org/jira/browse/MINIFICPP-2346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

James Guzman (Medel) updated MINIFICPP-2346:

Description: 
*NOTE:* I noticed you guys recently switched from 0.15 to 0.99, so at certain 
point, I will pull in lastest updates from MiNiFi C++ main branch on parent 
level and rebase or merge into my PR.

*Description of Issue:* Building MiNiFi C++ with just CMake is really slow. 
During the the build process, it clones all the external dependencies and then 
builds those external dependencies while its building MiNiFi CPP. Sometimes the 
MiNiFi CPP build fails too when external lib dependencies fail to download or 
fail to build. It would be faster to build MiNiFi C++ with the external 
dependencies already preinstalled.

 

*Potential Solution:* Integrate *conan version 2* into MiNiFi C++ project to 
enable support for using *conan install* to install all or most of MiNiFi's 
external dependencies, using *conan install --build=missing* to build them as 
prebuilt binary conan packages, upload them to conancenter with {*}conan 
upload{*}, and then run cmake generate to generate the appropriate build files 
for the OS, then run make to build MiNiFi CPP. Also conan has really good 
support for cross platform compatibility for Linux and Windows.  At this point 
because we already have most of MiNiFi CPP's external lib dependencies 
installed with conan, our CMake build will just focus on building the MiNiFi 
CPP code, enabling faster builds. Also I will need to account for backward 
compatibility support with the previous way of installing MiNiFi CPP external 
lib dependencies using CMake FetchContent_Declare(...) approach until 
conanfile.py approach covers installing all these dependencies. A valid 
verification of the MiNiFi C++ build would be to run GTESTs and if all the 
expected test cases pass, then it may be fair to conclude we would be ready to 
deprecate the slow CMake FetchContent_Declare(...) approach.

 

*Steps to Perform MiNiFi C++ Build Enhancement (Plan to Integrate into 
script(s)):*
 # Install conan version 2 into MiNiFi C++ dev environment (my preference is 
using docker container)
 # Create a conanfile.py file at root of MiNiFi C++ project where we run "conan 
install" or "conan install --build=missing" to install MiNiFi C++ external lib 
dependencies first. Then we run CMake to build MiNiFi C++.
 # Find all MiNiFi C++ find_package(...) CMake function calls and check if 
there is already supported conan packages on the conancenter, so we can install 
those packages.
 # Find all FetchContent_Declare(...) CMake function calls and check if there 
is already supported conan packages on the conancenter, so we can install those 
packages.
 # At this point we will have used conan version 2 C++ package manager to 
install almost all MiNiFi C++'s external dependencies.
 ## With Conan, we will have installed these MiNiFi external lib dependencies 
as prebuilt binary conan packages.
 ## Or we will have instructed conan to first build each C++ external lib 
dependency, upload each lib to conancenter.
 # Thus, conan manages installing MiNiFi C++'s external lib dependencies, so we 
can then run CMake to build MiNiFi faster.

 

Here is my draft PR for this Jira Ticket for your reference: 
[https://github.com/apache/nifi-minifi-cpp/pull/1775]

Here are the commands that I run:

 
{code:java}
cd $HOME/src/pipeline/nifi-minifi-cpp 

# NOTE: I also plan to add smoother integration of conan commands into MiNiFI 
CPP too
# For instance, I know you guys have your bootstrap scripts too
# You also have your github actions workflow scripts too
conan install . --output-folder=build 
-pr=$HOME/src/nifi-minifi-cpp/etc/build/conan/profiles/release-linux  

# install conan packages and build C++ for minifi using Conan & CMake, we run 
this way, so conan's tc.cache_variables
# are like passing -D{minifi_option} upon running cmake generate, then make 
to build MiNiFi C++
conan build . --output-folder=build 
-pr=/home/bizon/src/jam-repo/main/etc/build/conan/profiles/release-linux{code}
 

*UPDATE(May 14, 2024):* Updated due date to May 19, 2024 since the refactoring 
of MiNiFi CPP build infra over to conan version 2 is taking longer than 
expected. The good news is that I believe I have more than half of MiNiFi 
external libraries now managed by conan with backward compatibility for 
standalone CMake approach.

 

*UPDATE (May 13, 2024)* - 
*[22205c9|https://github.com/apache/nifi-minifi-cpp/pull/1775/commits/22205c98cef30ee1ebe252880ae113724589bd78]:*
 Fetch and Pulled latest upstream main branch of parent MiNiFi to my fork main 
branch of MiNiFi repo; thus, merged in the latest updates from main branch of 
parent MiNiFi to my MINIFICPP-2346 branch. Resolved conflicts between a few 
CMake files coming from Date.cmake and BundledRocksdb.cmake files. For 
standalone CMake build appraoch, 

[jira] [Updated] (MINIFICPP-2346) Speed up MiNiFi Build with Conan package manager (C++)

2024-05-14 Thread James Guzman (Medel) (Jira)


 [ 
https://issues.apache.org/jira/browse/MINIFICPP-2346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

James Guzman (Medel) updated MINIFICPP-2346:

Description: 
*NOTE:* I noticed you guys recently switched from 0.15 to 0.99, so at certain 
point, I will pull in lastest updates from MiNiFi C++ main branch on parent 
level and rebase or merge into my PR.

*Description of Issue:* Building MiNiFi C++ with just CMake is really slow. 
During the the build process, it clones all the external dependencies and then 
builds those external dependencies while its building MiNiFi CPP. Sometimes the 
MiNiFi CPP build fails too when external lib dependencies fail to download or 
fail to build. It would be faster to build MiNiFi C++ with the external 
dependencies already preinstalled.

 

*Potential Solution:* Integrate *conan version 2* into MiNiFi C++ project to 
enable support for using *conan install* to install all or most of MiNiFi's 
external dependencies, using *conan install --build=missing* to build them as 
prebuilt binary conan packages, upload them to conancenter with {*}conan 
upload{*}, and then run cmake generate to generate the appropriate build files 
for the OS, then run make to build MiNiFi CPP. Also conan has really good 
support for cross platform compatibility for Linux and Windows.  At this point 
because we already have most of MiNiFi CPP's external lib dependencies 
installed with conan, our CMake build will just focus on building the MiNiFi 
CPP code, enabling faster builds. Also I will need to account for backward 
compatibility support with the previous way of installing MiNiFi CPP external 
lib dependencies using CMake FetchContent_Declare(...) approach until 
conanfile.py approach covers installing all these dependencies. A valid 
verification of the MiNiFi C++ build would be to run GTESTs and if all the 
expected test cases pass, then it may be fair to conclude we would be ready to 
deprecate the slow CMake FetchContent_Declare(...) approach.

 

*Steps to Perform MiNiFi C++ Build Enhancement (Plan to Integrate into 
script(s)):*
 # Install conan version 2 into MiNiFi C++ dev environment (my preference is 
using docker container)
 # Create a conanfile.py file at root of MiNiFi C++ project where we run "conan 
install" or "conan install --build=missing" to install MiNiFi C++ external lib 
dependencies first. Then we run CMake to build MiNiFi C++.
 # Find all MiNiFi C++ find_package(...) CMake function calls and check if 
there is already supported conan packages on the conancenter, so we can install 
those packages.
 # Find all FetchContent_Declare(...) CMake function calls and check if there 
is already supported conan packages on the conancenter, so we can install those 
packages.
 # At this point we will have used conan version 2 C++ package manager to 
install almost all MiNiFi C++'s external dependencies.
 ## With Conan, we will have installed these MiNiFi external lib dependencies 
as prebuilt binary conan packages.
 ## Or we will have instructed conan to first build each C++ external lib 
dependency, upload each lib to conancenter.
 # Thus, conan manages installing MiNiFi C++'s external lib dependencies, so we 
can then run CMake to build MiNiFi faster.

 

Here is my draft PR for this Jira Ticket for your reference: 
[https://github.com/apache/nifi-minifi-cpp/pull/1775]

Here are the commands that I run:

 
{code:java}
cd $HOME/src/pipeline/nifi-minifi-cpp 

# NOTE: I also plan to add smoother integration of conan commands into MiNiFI 
CPP too
# For instance, I know you guys have your bootstrap scripts too
# You also have your github actions workflow scripts too
conan install . --output-folder=build 
-pr=$HOME/src/nifi-minifi-cpp/etc/build/conan/profiles/release-linux  

# install conan packages and build C++ for minifi using Conan & CMake, we run 
this way, so conan's tc.cache_variables
# are like passing -D{minifi_option} upon running cmake generate, then make 
to build MiNiFi C++
conan build . --output-folder=build 
-pr=/home/bizon/src/jam-repo/main/etc/build/conan/profiles/release-linux{code}
 

 

*UPDATE (May 13, 2024)* - 
[59ffefb|https://github.com/apache/nifi-minifi-cpp/pull/1775/commits/59ffefb1caf662874054f55f7171692c6b9d345f]:
 {*}Enabled LIBRDKAFKA & GRPC_FOR_LOKI MiNiFi C++ extension using Conan & 
CMake{*}, *Improved Running Python Maven Executable for JNI & SFTP Test Server 
for conan approach* and {*}Improved standalone CMake approach for building 
PcapPlusPlus with its dependency on LibPcap{*}. LIBRDKAFKA required creating a 
librdkafka conan package passing similar CMake definitions to it to build it 
like we did for standalone CMake approach. GRPC_FOR_LOKI required creating a 
grpc conan package that uses minifi's openssl and libsystemd conan packages. In 
both LIBRDKAFKA & GRPC_FOR_LOKI, I updated their .cmake Bundles to account for 
the condition of using their conan package, 

[jira] [Updated] (MINIFICPP-2346) Speed up MiNiFi Build with Conan package manager (C++)

2024-05-14 Thread James Guzman (Medel) (Jira)


 [ 
https://issues.apache.org/jira/browse/MINIFICPP-2346?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

James Guzman (Medel) updated MINIFICPP-2346:

Due Date: 19/May/24  (was: 12/May/24)

> Speed up MiNiFi Build with Conan package manager (C++)
> --
>
> Key: MINIFICPP-2346
> URL: https://issues.apache.org/jira/browse/MINIFICPP-2346
> Project: Apache NiFi MiNiFi C++
>  Issue Type: Improvement
>Affects Versions: 0.15.0
> Environment: Ubuntu 22.04
>Reporter: James Guzman (Medel)
>Assignee: James Guzman (Medel)
>Priority: Minor
>  Labels: build, conan, ubuntu
> Attachments: minifi_cpp_cmake_standalone_build_7eb8beb.log, 
> minifi_cpp_conan_build_2d1b884.log, minifi_cpp_conan_build_7eb8beb.log
>
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> *NOTE:* I noticed you guys recently switched from 0.15 to 0.99, so at certain 
> point, I will pull in lastest updates from MiNiFi C++ main branch on parent 
> level and rebase or merge into my PR.
> *Description of Issue:* Building MiNiFi C++ with just CMake is really slow. 
> During the the build process, it clones all the external dependencies and 
> then builds those external dependencies while its building MiNiFi CPP. 
> Sometimes the MiNiFi CPP build fails too when external lib dependencies fail 
> to download or fail to build. It would be faster to build MiNiFi C++ with the 
> external dependencies already preinstalled.
>  
> *Potential Solution:* Integrate *conan version 2* into MiNiFi C++ project to 
> enable support for using *conan install* to install all or most of MiNiFi's 
> external dependencies, using *conan install --build=missing* to build them as 
> prebuilt binary conan packages, upload them to conancenter with {*}conan 
> upload{*}, and then run cmake generate to generate the appropriate build 
> files for the OS, then run make to build MiNiFi CPP. Also conan has really 
> good support for cross platform compatibility for Linux and Windows.  At this 
> point because we already have most of MiNiFi CPP's external lib dependencies 
> installed with conan, our CMake build will just focus on building the MiNiFi 
> CPP code, enabling faster builds. Also I will need to account for backward 
> compatibility support with the previous way of installing MiNiFi CPP external 
> lib dependencies using CMake FetchContent_Declare(...) approach until 
> conanfile.py approach covers installing all these dependencies. A valid 
> verification of the MiNiFi C++ build would be to run GTESTs and if all the 
> expected test cases pass, then it may be fair to conclude we would be ready 
> to deprecate the slow CMake FetchContent_Declare(...) approach.
>  
> *Steps to Perform MiNiFi C++ Build Enhancement (Plan to Integrate into 
> script(s)):*
>  # Install conan version 2 into MiNiFi C++ dev environment (my preference is 
> using docker container)
>  # Create a conanfile.py file at root of MiNiFi C++ project where we run 
> "conan install" or "conan install --build=missing" to install MiNiFi C++ 
> external lib dependencies first. Then we run CMake to build MiNiFi C++.
>  # Find all MiNiFi C++ find_package(...) CMake function calls and check if 
> there is already supported conan packages on the conancenter, so we can 
> install those packages.
>  # Find all FetchContent_Declare(...) CMake function calls and check if there 
> is already supported conan packages on the conancenter, so we can install 
> those packages.
>  # At this point we will have used conan version 2 C++ package manager to 
> install almost all MiNiFi C++'s external dependencies.
>  ## With Conan, we will have installed these MiNiFi external lib dependencies 
> as prebuilt binary conan packages.
>  ## Or we will have instructed conan to first build each C++ external lib 
> dependency, upload each lib to conancenter.
>  # Thus, conan manages installing MiNiFi C++'s external lib dependencies, so 
> we can then run CMake to build MiNiFi faster.
>  
> Here is my draft PR for this Jira Ticket for your reference: 
> [https://github.com/apache/nifi-minifi-cpp/pull/1775]
> Here are the commands that I run:
>  
> {code:java}
> cd $HOME/src/pipeline/nifi-minifi-cpp 
> # NOTE: I also plan to add smoother integration of conan commands into MiNiFI 
> CPP too
> # For instance, I know you guys have your bootstrap scripts too
> # You also have your github actions workflow scripts too
> conan install . --output-folder=build 
> -pr=$HOME/src/nifi-minifi-cpp/etc/build/conan/profiles/release-linux  
> # install conan packages and build C++ for minifi using Conan & CMake, we run 
> this way, so conan's tc.cache_variables
> # are like passing -D{minifi_option} upon running cmake generate, then 
> make to build MiNiFi C++
> conan build . --output-folder=build 
> 

Re: [PR] [NIFI-13207] page headers and refresh containers are consistently pos… [nifi]

2024-05-14 Thread via GitHub


scottyaslan commented on PR #8804:
URL: https://github.com/apache/nifi/pull/8804#issuecomment-2111436636

   Update:
   
   All page headers should have 1.25rem padding from the navigation toolbar at 
the top of each page and they should have 1.25rem padding below:
   
   https://github.com/apache/nifi/assets/6797571/e3c79985-f9cf-4a86-95fa-e7f96f93a1f2;>
   
   Any text above the filter form fields should have 0.75rem padding below:
   
   https://github.com/apache/nifi/assets/6797571/279de58a-02ea-47f8-8f8c-f327b9c2543d;>
   
   Any hint text should have 0.25rem padding above and 0.5rem padding below:
   
   https://github.com/apache/nifi/assets/6797571/a9844a4e-971b-438a-bf62-acdf48fb327b;>
   https://github.com/apache/nifi/assets/6797571/f9b909a5-818f-49cb-8d80-390661b70a90;>
   
   For the footer (last updated etc) it should have 0.5rem padding above and 
1.25rem padding below:
   
   https://github.com/apache/nifi/assets/6797571/8a9153d9-ea6c-4dbf-886b-d772e205f922;>
   https://github.com/apache/nifi/assets/6797571/85bba52f-5115-4cde-b620-704bf566b218;>
   
   Tabs (not in dialogs) should have 1.25rem padding below:
   
   https://github.com/apache/nifi/assets/6797571/d08ebfc8-ad5c-4107-8ff9-8ca82f18c016;>
   


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [NIFI-13207] page headers and refresh containers are consistently pos… [nifi]

2024-05-14 Thread via GitHub


scottyaslan commented on code in PR #8804:
URL: https://github.com/apache/nifi/pull/8804#discussion_r1600842421


##
nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi/src/app/pages/provenance/ui/provenance-event-listing/provenance-event-table/provenance-event-table.component.html:
##
@@ -17,7 +17,7 @@
 
 
 
-
+

Review Comment:
   Ok, I have updated the layouts to display filter hint text below the form 
fields.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13241) [BUG] Escape button does not close extension creation dialogs

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13241:
---
Description: This issue affects all creation dialogs that set focus to an 
input.

> [BUG] Escape button does not close extension creation dialogs
> -
>
> Key: NIFI-13241
> URL: https://issues.apache.org/jira/browse/NIFI-13241
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Scott Aslan
>Priority: Major
>
> This issue affects all creation dialogs that set focus to an input.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13241) [BUG] Escape button does not close extension creation dialogs

2024-05-14 Thread Scott Aslan (Jira)
Scott Aslan created NIFI-13241:
--

 Summary: [BUG] Escape button does not close extension creation 
dialogs
 Key: NIFI-13241
 URL: https://issues.apache.org/jira/browse/NIFI-13241
 Project: Apache NiFi
  Issue Type: Sub-task
Reporter: Scott Aslan






--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-12595 Introduce 'Track Last Listing Time' mode to ListedEntityTracker [nifi]

2024-05-14 Thread via GitHub


github-actions[bot] commented on PR #8244:
URL: https://github.com/apache/nifi/pull/8244#issuecomment-2111361400

   Automated review is marking this PR as stale due to lack of updates in the 
past four months. This PR will be closed in 15 days if the stale label is not 
removed. This stale label and automated closure does not indicate a judgement 
of the PR, just lack of reviewer bandwidth and helps us keep the PR queue more 
manageable.  If you would like this PR re-opened you can do so and a committer 
can remove the stale label.  Or you can open a new PR.  Try to help review 
other PRs to increase PR review bandwidth which in turn helps yours.


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13197) Clicking outside of dialog cancels edit

2024-05-14 Thread Joe Witt (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Joe Witt updated NIFI-13197:

Fix Version/s: 2.0.0-M4

> Clicking outside of dialog cancels edit
> ---
>
> Key: NIFI-13197
> URL: https://issues.apache.org/jira/browse/NIFI-13197
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Rob Fellows
>Priority: Major
> Fix For: 2.0.0-M4
>
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> Default behavior of dialog is to support closing when the user clicks off. 
> Incidental clicks could result in the loss of unsaved changes. We should 
> consider
>  * Always preventing closing
>  * Only prevent closing for Edit dialogs
>  * Only prevent closing when dialogs contain a dirty form



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13197) Clicking outside of dialog cancels edit

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13197:
---
Resolution: Fixed
Status: Resolved  (was: Patch Available)

> Clicking outside of dialog cancels edit
> ---
>
> Key: NIFI-13197
> URL: https://issues.apache.org/jira/browse/NIFI-13197
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Rob Fellows
>Priority: Major
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> Default behavior of dialog is to support closing when the user clicks off. 
> Incidental clicks could result in the loss of unsaved changes. We should 
> consider
>  * Always preventing closing
>  * Only prevent closing for Edit dialogs
>  * Only prevent closing when dialogs contain a dirty form



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-13197) Clicking outside of dialog cancels edit

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-13197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846456#comment-17846456
 ] 

ASF subversion and git services commented on NIFI-13197:


Commit 63a12b06b5f9dddabe5a44874194c745e067bcb5 in nifi's branch 
refs/heads/main from Rob Fellows
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=63a12b06b5 ]

[NIFI-13197] - Clicking outside of a dialog no longer closes the dialog (#8811)

* [NIFI-13197] - Clicking outside of a dialog no longer closes the dialog. 
Also, escape will only close the dialog if the underlying form is not dirty.

* add license header

* allow ESC to close even if the for is dirty

This closes #8811

> Clicking outside of dialog cancels edit
> ---
>
> Key: NIFI-13197
> URL: https://issues.apache.org/jira/browse/NIFI-13197
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Rob Fellows
>Priority: Major
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> Default behavior of dialog is to support closing when the user clicks off. 
> Incidental clicks could result in the loss of unsaved changes. We should 
> consider
>  * Always preventing closing
>  * Only prevent closing for Edit dialogs
>  * Only prevent closing when dialogs contain a dirty form



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] [NIFI-13197] - Clicking outside of a dialog no longer closes the dialog [nifi]

2024-05-14 Thread via GitHub


scottyaslan merged PR #8811:
URL: https://github.com/apache/nifi/pull/8811


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [NIFI-13197] - Clicking outside of a dialog no longer closes the dialog [nifi]

2024-05-14 Thread via GitHub


scottyaslan commented on code in PR #8811:
URL: https://github.com/apache/nifi/pull/8811#discussion_r1600724133


##
nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi/src/app/ui/common/close-on-escape-dialog/close-on-escape-dialog.component.ts:
##
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { Component, inject } from '@angular/core';
+import { CommonModule } from '@angular/common';
+import { filter } from 'rxjs';
+import { MatDialogRef } from '@angular/material/dialog';
+import { takeUntilDestroyed } from '@angular/core/rxjs-interop';
+
+@Component({
+selector: 'close-on-escape-dialog',
+standalone: true,
+imports: [CommonModule],
+template: ''
+})
+export abstract class CloseOnEscapeDialog {
+private dialogRef: MatDialogRef = 
inject(MatDialogRef);
+
+protected constructor() {
+if (this.dialogRef) {
+this.dialogRef
+.keydownEvents()
+.pipe(
+filter((event: KeyboardEvent) => event.key === 'Escape'),
+takeUntilDestroyed()
+)
+.subscribe(() => {
+if (!this.isDirty()) {
+this.dialogRef.close();
+}
+});
+}

Review Comment:
   Let's just update the UX to match legacy nifi.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [NIFI-13082] Created SplitPcap processor, Pcap supporting class, and … [nifi]

2024-05-14 Thread via GitHub


JackHintonSmartDCSIT commented on code in PR #8691:
URL: https://github.com/apache/nifi/pull/8691#discussion_r1600715641


##
nifi-extension-bundles/nifi-network-bundle/nifi-network-processors/src/main/java/org/apache/nifi/processors/network/SplitPCAP.java:
##
@@ -0,0 +1,203 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.network;
+
+import org.apache.nifi.components.PropertyDescriptor;
+import org.apache.nifi.flowfile.FlowFile;
+import org.apache.nifi.annotation.lifecycle.OnScheduled;
+import org.apache.nifi.annotation.behavior.WritesAttribute;
+import org.apache.nifi.annotation.documentation.CapabilityDescription;
+import org.apache.nifi.annotation.documentation.Tags;
+import org.apache.nifi.processor.AbstractProcessor;
+import org.apache.nifi.processor.ProcessContext;
+import org.apache.nifi.processor.ProcessSession;
+import org.apache.nifi.processor.Relationship;
+import org.apache.nifi.processor.util.StandardValidators;
+import org.apache.nifi.processors.network.util.PCAP;
+import org.apache.nifi.processors.network.util.PCAP.ByteBufferInterface;
+import org.apache.nifi.processors.network.util.PCAP.Packet;
+
+import java.io.ByteArrayOutputStream;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+@Tags({"PCAP", "Splitter", "Network", "Packet", "Capture", "Wireshark", 
"TShark"})
+@CapabilityDescription("Splits a pcap file into multiple pcap files based on a 
maximum size.")
+@WritesAttribute(attribute = "ERROR_REASON", description = "The reason the 
flowfile was sent to the failure relationship.")
+
+public class SplitPCAP extends AbstractProcessor {
+

Review Comment:
   I've changed it to `protected` and that seems to work.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (NIFI-13240) Create Marker Interface for AbstractSingleAttributeControllerService

2024-05-14 Thread Jim Steinebrey (Jira)
Jim Steinebrey created NIFI-13240:
-

 Summary: Create Marker Interface for 
AbstractSingleAttributeControllerService
 Key: NIFI-13240
 URL: https://issues.apache.org/jira/browse/NIFI-13240
 Project: Apache NiFi
  Issue Type: Improvement
Reporter: Jim Steinebrey






--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-8134) DataTypeUtils.toRecord methods do not recursively convert Maps into Records

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-8134?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846449#comment-17846449
 ] 

ASF subversion and git services commented on NIFI-8134:
---

Commit 2d112871db40c8f599d31974d14dd4a2227bf7da in nifi's branch 
refs/heads/main from Chris Sampson
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=2d112871db ]

NIFI-8134 allow unescapeJson Record Path function to recursively convert Maps 
to Records (#7745)

* NIFI-8134 recursively convert Java Objects to NiFi Records

> DataTypeUtils.toRecord methods do not recursively convert Maps into Records
> ---
>
> Key: NIFI-8134
> URL: https://issues.apache.org/jira/browse/NIFI-8134
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Core Framework
>Affects Versions: 1.11.4
>Reporter: Chris Sampson
>Assignee: Chris Sampson
>Priority: Major
>  Time Spent: 3h
>  Remaining Estimate: 0h
>
> Given a java Map that contains one or more Maps as values (optionally nested 
> within arrays), the DataTypeUtils.toRecord method should convert the child 
> Maps to Records before converting the to level Map.
> This assumes the associated schema for the data represents these objects as 
> Records (including as part of a Choice of Array type).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-8134) DataTypeUtils.toRecord methods do not recursively convert Maps into Records

2024-05-14 Thread Mark Payne (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-8134?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mark Payne updated NIFI-8134:
-
Fix Version/s: 2.0.0-M4
   Resolution: Fixed
   Status: Resolved  (was: Patch Available)

> DataTypeUtils.toRecord methods do not recursively convert Maps into Records
> ---
>
> Key: NIFI-8134
> URL: https://issues.apache.org/jira/browse/NIFI-8134
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Core Framework
>Affects Versions: 1.11.4
>Reporter: Chris Sampson
>Assignee: Chris Sampson
>Priority: Major
> Fix For: 2.0.0-M4
>
>  Time Spent: 3h
>  Remaining Estimate: 0h
>
> Given a java Map that contains one or more Maps as values (optionally nested 
> within arrays), the DataTypeUtils.toRecord method should convert the child 
> Maps to Records before converting the to level Map.
> This assumes the associated schema for the data represents these objects as 
> Records (including as part of a Choice of Array type).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-8134 allow unescapeJson Record Path function to recursively convert Maps to Records [nifi]

2024-05-14 Thread via GitHub


markap14 merged PR #7745:
URL: https://github.com/apache/nifi/pull/7745


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] NIFI-8134 allow unescapeJson Record Path function to recursively convert Maps to Records [nifi]

2024-05-14 Thread via GitHub


markap14 commented on PR #7745:
URL: https://github.com/apache/nifi/pull/7745#issuecomment-249151

   Thanks for the update @ChrisSamo632 all looks good to me at this point. 
Sorry this has taken so long, and thanks for sticking with it! +1 will merge to 
main


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (MINIFICPP-2373) ConsumeJournald is broken with 'Process Old Messages'=false

2024-05-14 Thread Marton Szasz (Jira)
Marton Szasz created MINIFICPP-2373:
---

 Summary: ConsumeJournald is broken with 'Process Old 
Messages'=false
 Key: MINIFICPP-2373
 URL: https://issues.apache.org/jira/browse/MINIFICPP-2373
 Project: Apache NiFi MiNiFi C++
  Issue Type: Bug
Reporter: Marton Szasz


{{[2024-05-14 22:58:24.171] [org::apache::nifi::minifi::SchedulingAgent] 
[warning] Caught Exception during SchedulingAgent::onTrigger of processor 
ConsumeJournald (uuid: fa39ba8a-6ea8-4c4d-90bd-b8da329f0f6e), type: 
N3org6apache4nifi6minifi20SystemErrorExceptionE, what: sd_journal_get_cursor: 
Cannot assign requested address}}
 
 Maybe libsystemd broke API?



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-12896: Add Endpoint Override URL property for PutSNS processor [nifi]

2024-05-14 Thread via GitHub


exceptionfactory commented on code in PR #8529:
URL: https://github.com/apache/nifi/pull/8529#discussion_r1600636094


##
nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/test/java/org/apache/nifi/processors/aws/sns/ITPutSNS.java:
##
@@ -0,0 +1,119 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.aws.sns;
+
+import org.apache.nifi.processor.Processor;
+import org.apache.nifi.processors.aws.testutil.AuthUtils;
+import org.apache.nifi.processors.aws.v2.AbstractAwsProcessor;
+import org.apache.nifi.util.TestRunner;
+import org.apache.nifi.util.TestRunners;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.testcontainers.containers.localstack.LocalStackContainer;
+import org.testcontainers.utility.DockerImageName;
+import software.amazon.awssdk.auth.credentials.AwsBasicCredentials;
+import software.amazon.awssdk.auth.credentials.StaticCredentialsProvider;
+import software.amazon.awssdk.regions.Region;
+import software.amazon.awssdk.services.sns.SnsClient;
+import software.amazon.awssdk.services.sns.model.CreateTopicRequest;
+import software.amazon.awssdk.services.sns.model.CreateTopicResponse;
+
+import java.io.IOException;
+import java.nio.file.Paths;
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Provides integration level testing with actual AWS S3 resources for {@link 
PutSNS} and requires additional configuration and resources to work.
+ */
+public class ITPutSNS {
+
+private static final DockerImageName localstackImage = 
DockerImageName.parse("localstack/localstack:latest");
+
+private static final LocalStackContainer localstack = new 
LocalStackContainer(localstackImage)
+.withServices(LocalStackContainer.Service.SNS);
+
+private static final String CREDENTIALS_FILE = 
"src/test/resources/mock-aws-credentials.properties";
+private static String topicARN;
+private static SnsClient client;
+
+@BeforeAll
+public static void setup() throws InterruptedException {
+System.setProperty("software.amazon.awssdk.http.service.impl", 
"software.amazon.awssdk.http.urlconnection.UrlConnectionSdkHttpService");

Review Comment:
   Thanks!



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] NIFI-12896: Add Endpoint Override URL property for PutSNS processor [nifi]

2024-05-14 Thread via GitHub


sujkm commented on code in PR #8529:
URL: https://github.com/apache/nifi/pull/8529#discussion_r1600620568


##
nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/test/java/org/apache/nifi/processors/aws/sns/ITPutSNS.java:
##
@@ -0,0 +1,119 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.aws.sns;
+
+import org.apache.nifi.processor.Processor;
+import org.apache.nifi.processors.aws.testutil.AuthUtils;
+import org.apache.nifi.processors.aws.v2.AbstractAwsProcessor;
+import org.apache.nifi.util.TestRunner;
+import org.apache.nifi.util.TestRunners;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.testcontainers.containers.localstack.LocalStackContainer;
+import org.testcontainers.utility.DockerImageName;
+import software.amazon.awssdk.auth.credentials.AwsBasicCredentials;
+import software.amazon.awssdk.auth.credentials.StaticCredentialsProvider;
+import software.amazon.awssdk.regions.Region;
+import software.amazon.awssdk.services.sns.SnsClient;
+import software.amazon.awssdk.services.sns.model.CreateTopicRequest;
+import software.amazon.awssdk.services.sns.model.CreateTopicResponse;
+
+import java.io.IOException;
+import java.nio.file.Paths;
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Provides integration level testing with actual AWS S3 resources for {@link 
PutSNS} and requires additional configuration and resources to work.
+ */
+public class ITPutSNS {
+
+private static final DockerImageName localstackImage = 
DockerImageName.parse("localstack/localstack:latest");
+
+private static final LocalStackContainer localstack = new 
LocalStackContainer(localstackImage)
+.withServices(LocalStackContainer.Service.SNS);
+
+private static final String CREDENTIALS_FILE = 
"src/test/resources/mock-aws-credentials.properties";
+private static String topicARN;
+private static SnsClient client;
+
+@BeforeAll
+public static void setup() throws InterruptedException {
+System.setProperty("software.amazon.awssdk.http.service.impl", 
"software.amazon.awssdk.http.urlconnection.UrlConnectionSdkHttpService");

Review Comment:
   It was not required it has been removed.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (NIFI-13239) Prevent routing away from a dirty form without prompting the user

2024-05-14 Thread Rob Fellows (Jira)
Rob Fellows created NIFI-13239:
--

 Summary: Prevent routing away from a dirty form without prompting 
the user
 Key: NIFI-13239
 URL: https://issues.apache.org/jira/browse/NIFI-13239
 Project: Apache NiFi
  Issue Type: Sub-task
  Components: Core UI
Reporter: Rob Fellows


If users make changes to a configuration component and hit the browser back 
button or refresh button, they aren't prompted about saving unfinished changes. 
This is a potential for data loss that might be solvable with canDeactivate 
guards.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-12896: Add Endpoint Override URL property for PutSNS processor [nifi]

2024-05-14 Thread via GitHub


exceptionfactory commented on code in PR #8529:
URL: https://github.com/apache/nifi/pull/8529#discussion_r1600591314


##
nifi-extension-bundles/nifi-aws-bundle/nifi-aws-processors/src/test/java/org/apache/nifi/processors/aws/sns/ITPutSNS.java:
##
@@ -0,0 +1,119 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.aws.sns;
+
+import org.apache.nifi.processor.Processor;
+import org.apache.nifi.processors.aws.testutil.AuthUtils;
+import org.apache.nifi.processors.aws.v2.AbstractAwsProcessor;
+import org.apache.nifi.util.TestRunner;
+import org.apache.nifi.util.TestRunners;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.testcontainers.containers.localstack.LocalStackContainer;
+import org.testcontainers.utility.DockerImageName;
+import software.amazon.awssdk.auth.credentials.AwsBasicCredentials;
+import software.amazon.awssdk.auth.credentials.StaticCredentialsProvider;
+import software.amazon.awssdk.regions.Region;
+import software.amazon.awssdk.services.sns.SnsClient;
+import software.amazon.awssdk.services.sns.model.CreateTopicRequest;
+import software.amazon.awssdk.services.sns.model.CreateTopicResponse;
+
+import java.io.IOException;
+import java.nio.file.Paths;
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * Provides integration level testing with actual AWS S3 resources for {@link 
PutSNS} and requires additional configuration and resources to work.
+ */
+public class ITPutSNS {
+
+private static final DockerImageName localstackImage = 
DockerImageName.parse("localstack/localstack:latest");
+
+private static final LocalStackContainer localstack = new 
LocalStackContainer(localstackImage)
+.withServices(LocalStackContainer.Service.SNS);
+
+private static final String CREDENTIALS_FILE = 
"src/test/resources/mock-aws-credentials.properties";
+private static String topicARN;
+private static SnsClient client;
+
+@BeforeAll
+public static void setup() throws InterruptedException {
+System.setProperty("software.amazon.awssdk.http.service.impl", 
"software.amazon.awssdk.http.urlconnection.UrlConnectionSdkHttpService");

Review Comment:
   Is this system property required? If it is needed, this property should be 
removed in the `@AfterAll` method.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] NIFI-12411: Update PublishAMQP to read AMQP headers value from FlowFile attributes and `amq$headers` string [nifi]

2024-05-14 Thread via GitHub


exceptionfactory commented on PR #8105:
URL: https://github.com/apache/nifi/pull/8105#issuecomment-2111054120

   > @exceptionfactory @joewitt
   > 
   > I have added one more change, which will allow users to set amqp 
processor's password via env variables. I think this will be helpful in cases 
where we use AMQP processors in multiple NiFI instances. Instead of copying and 
pasting different passwords in processor properties, we can reference the env 
variable via its name. Let me know your thoughts on it. Thanks.
   
   Thanks for calling out this particular change @umarhussain15. Given the 
scope of the current pull request, I recommend considering that change 
separately.
   
   In general, sensitive properties such as passwords do not support reading 
from environment variables. The reason is that environment variables are also 
accessible through Expression Language, so they are not necessarily secure. The 
use case is still supported through Parameter Contexts and Parameter Providers, 
where the Environment Variable Parameter Provider can be used. This is an 
additional layer of indirection, but it follows the general convention for 
other sensitive properties.


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13211) Decouple UIs from old NiFi UI

2024-05-14 Thread Rob Fellows (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rob Fellows updated NIFI-13211:
---
Fix Version/s: 2.0.0-M4
   Resolution: Fixed
   Status: Resolved  (was: Patch Available)

> Decouple UIs from old NiFi UI
> -
>
> Key: NIFI-13211
> URL: https://issues.apache.org/jira/browse/NIFI-13211
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
> Fix For: 2.0.0-M4
>
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> NiFi is comprised of numerous UIs that all get deployed at start up. These 
> UIs piggybacked on some shared stylesheets, components, and dependencies by 
> directly loading them through a relative path. In preparation for moving away 
> from the existing UI, these UIs are being updated to bundle these items 
> directly. In time, these UIs will all be modernized as well so this is just a 
> transitional step in that process.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-13211) Decouple UIs from old NiFi UI

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-13211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846411#comment-17846411
 ] 

ASF subversion and git services commented on NIFI-13211:


Commit 3a786cee23313ae365845fc796542469e035e48a in nifi's branch 
refs/heads/main from Matt Gilman
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=3a786cee23 ]

NIFI-13211: Decoupling other UIs from exiting NiFi UI (#8828)

* NIFI-13211:
- Decoupling the content viewer from the NiFi UI which is currently pulling in 
shared dependencies, scripts, styles, and images.

* NIFI-13211:
- Decoupling nifi documentation from the NiFi UI which is currently pulling in 
shared dependencies, scripts, styles, and images.

* NIFI-13211:
- Decoupling nifi image viewer from the NiFi UI which is currently pulling in 
shared dependencies, scripts, styles, and images.

* NIFI-13211:
- Decoupling nifi standard content viewer from the NiFi UI which is currently 
pulling in shared dependencies, scripts, styles, and images.

* NIFI-13211:
- Decoupling nifi jolt transform json from the NiFi UI which is currently 
pulling in shared dependencies, scripts, styles, and images.

* NIFI-13211:
- Decoupling nifi update attribute from the NiFi UI which is currently pulling 
in shared dependencies, scripts, styles, and images.

* NIFI-13211:
- Updating rat config.

* NIFI-13211:
- Bumping dependency check timeout.

* NIFI-13211:
- Since this page is included through the context viewer, the dependency 
imports must directly reference through its context path.

* NIFI-13211:
- Since this page is included through the context viewer, the dependency 
imports must directly reference through its context path.

This closes #8828 

> Decouple UIs from old NiFi UI
> -
>
> Key: NIFI-13211
> URL: https://issues.apache.org/jira/browse/NIFI-13211
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> NiFi is comprised of numerous UIs that all get deployed at start up. These 
> UIs piggybacked on some shared stylesheets, components, and dependencies by 
> directly loading them through a relative path. In preparation for moving away 
> from the existing UI, these UIs are being updated to bundle these items 
> directly. In time, these UIs will all be modernized as well so this is just a 
> transitional step in that process.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-13211: Decoupling other UIs from exiting NiFi UI [nifi]

2024-05-14 Thread via GitHub


rfellows merged PR #8828:
URL: https://github.com/apache/nifi/pull/8828


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13238) Checkstyle - rules for whitespace

2024-05-14 Thread Pierre Villard (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13238?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Pierre Villard updated NIFI-13238:
--
Status: Patch Available  (was: Open)

> Checkstyle - rules for whitespace
> -
>
> Key: NIFI-13238
> URL: https://issues.apache.org/jira/browse/NIFI-13238
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Tools and Build
>Reporter: Pierre Villard
>Assignee: Pierre Villard
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Adding the following rules:
>  * WhitespaceAfter
>  * NoWhitespaceAfter
>  * NoWhitespaceBefore
>  * WhitespaceAround



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[PR] NIFI-13238 - Adding checkstyle rules for whitespaces [nifi]

2024-05-14 Thread via GitHub


pvillard31 opened a new pull request, #8832:
URL: https://github.com/apache/nifi/pull/8832

   # Summary
   
   [NIFI-13238](https://issues.apache.org/jira/browse/NIFI-13238) - Adding 
checkstyle rules for whitespaces
   
   # Tracking
   
   Please complete the following tracking steps prior to pull request creation.
   
   ### Issue Tracking
   
   - [ ] [Apache NiFi Jira](https://issues.apache.org/jira/browse/NIFI) issue 
created
   
   ### Pull Request Tracking
   
   - [ ] Pull Request title starts with Apache NiFi Jira issue number, such as 
`NIFI-0`
   - [ ] Pull Request commit message starts with Apache NiFi Jira issue number, 
as such `NIFI-0`
   
   ### Pull Request Formatting
   
   - [ ] Pull Request based on current revision of the `main` branch
   - [ ] Pull Request refers to a feature branch with one commit containing 
changes
   
   # Verification
   
   Please indicate the verification steps performed prior to pull request 
creation.
   
   ### Build
   
   - [ ] Build completed using `mvn clean install -P contrib-check`
 - [ ] JDK 21
   
   ### UI Contributions
   
   - [ ] NiFi is modernizing its UI. Any contributions that update the [current 
UI](https://github.com/apache/nifi/tree/main/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui)
 also need to be implemented in the [new 
UI](https://github.com/apache/nifi/tree/main/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi).
  
   
   ### Licensing
   
   - [ ] New dependencies are compatible with the [Apache License 
2.0](https://apache.org/licenses/LICENSE-2.0) according to the [License 
Policy](https://www.apache.org/legal/resolved.html)
   - [ ] New dependencies are documented in applicable `LICENSE` and `NOTICE` 
files
   
   ### Documentation
   
   - [ ] Documentation formatting appears as expected in rendered files
   


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (NIFI-13238) Checkstyle - rules for whitespace

2024-05-14 Thread Pierre Villard (Jira)
Pierre Villard created NIFI-13238:
-

 Summary: Checkstyle - rules for whitespace
 Key: NIFI-13238
 URL: https://issues.apache.org/jira/browse/NIFI-13238
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Tools and Build
Reporter: Pierre Villard
Assignee: Pierre Villard


Adding the following rules:
 * WhitespaceAfter
 * NoWhitespaceAfter
 * NoWhitespaceBefore
 * WhitespaceAround



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13237) Checkstyle improvements

2024-05-14 Thread Pierre Villard (Jira)
Pierre Villard created NIFI-13237:
-

 Summary: Checkstyle improvements
 Key: NIFI-13237
 URL: https://issues.apache.org/jira/browse/NIFI-13237
 Project: Apache NiFi
  Issue Type: Epic
Reporter: Pierre Villard
Assignee: Pierre Villard


Epic to track the addition of some rules in the checkstyle configuration of the 
NiFi project.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13236) Issue when viewing and downloading content

2024-05-14 Thread Matt Gilman (Jira)
Matt Gilman created NIFI-13236:
--

 Summary: Issue when viewing and downloading content
 Key: NIFI-13236
 URL: https://issues.apache.org/jira/browse/NIFI-13236
 Project: Apache NiFi
  Issue Type: Bug
  Components: Core Framework
Reporter: Matt Gilman


When attempting to view or download content the flowfile's filename (from the 
filename attribute) is used as the proposed downloaded filename. These 
endpoints support gzip encoding. If the filename contains characters are 
invalid VCHAR based on RFC9110, the endpoint fails with a connection error. The 
log contains the following exception.

 
{noformat}
2024-05-14 14:52:40,290 ERROR [NiFi Web Server-138] 
o.g.j.server.ServerRuntime$Responder An I/O error has occurred while writing a 
response message entity to the container output stream.
org.glassfish.jersey.server.internal.process.MappableException: 
java.io.IOException: 
org.eclipse.jetty.http2.hpack.HpackException$SessionException: Could not hpack 
encode HTTP/2.0{s=200,h=11,cl=-1}
    at 
org.glassfish.jersey.server.internal.MappableExceptionWrapperInterceptor.aroundWriteTo(MappableExceptionWrapperInterceptor.java:67)
    at 
org.glassfish.jersey.message.internal.WriterInterceptorExecutor.proceed(WriterInterceptorExecutor.java:139)
    at 
org.glassfish.jersey.message.internal.MessageBodyFactory.writeTo(MessageBodyFactory.java:1116)
    at 
org.glassfish.jersey.server.ServerRuntime$Responder.writeResponse(ServerRuntime.java:678)
    at 
org.glassfish.jersey.server.ServerRuntime$Responder.processResponse(ServerRuntime.java:387)
    at 
org.glassfish.jersey.server.ServerRuntime$Responder.process(ServerRuntime.java:377)
    at org.glassfish.jersey.server.ServerRuntime$1.run(ServerRuntime.java:266)
    at org.glassfish.jersey.internal.Errors$1.call(Errors.java:248)
    at org.glassfish.jersey.internal.Errors$1.call(Errors.java:244)
    at org.glassfish.jersey.internal.Errors.process(Errors.java:292)
    at org.glassfish.jersey.internal.Errors.process(Errors.java:274)
    at org.glassfish.jersey.internal.Errors.process(Errors.java:244)
    at 
org.glassfish.jersey.process.internal.RequestScope.runInScope(RequestScope.java:266)
    at org.glassfish.jersey.server.ServerRuntime.process(ServerRuntime.java:242)
    at 
org.glassfish.jersey.server.ApplicationHandler.handle(ApplicationHandler.java:697)
    at 
org.glassfish.jersey.servlet.WebComponent.serviceImpl(WebComponent.java:394)
    at org.glassfish.jersey.servlet.WebComponent.service(WebComponent.java:346)
    at 
org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:358)
    at 
org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:312)
    at 
org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:205)
    at 
org.eclipse.jetty.ee10.servlet.ServletHolder$NotAsync.service(ServletHolder.java:1379)
    at 
org.eclipse.jetty.ee10.servlet.ServletHolder.handle(ServletHolder.java:736)
    at 
org.eclipse.jetty.ee10.servlet.ServletHandler$ChainEnd.doFilter(ServletHandler.java:1614)
    at 
org.springframework.security.web.FilterChainProxy.lambda$doFilterInternal$3(FilterChainProxy.java:231)
    at 
org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:365)
    at 
org.springframework.security.web.access.intercept.AuthorizationFilter.doFilter(AuthorizationFilter.java:100)
    at 
org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:374)
    at 
org.springframework.security.web.access.ExceptionTranslationFilter.doFilter(ExceptionTranslationFilter.java:126)
    at 
org.springframework.security.web.access.ExceptionTranslationFilter.doFilter(ExceptionTranslationFilter.java:120)
    at 
org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:374)
    at 
org.apache.nifi.web.security.log.AuthenticationUserFilter.doFilterInternal(AuthenticationUserFilter.java:57)
    at 
org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:116)
    at 
org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:374)
    at 
org.springframework.security.web.authentication.AnonymousAuthenticationFilter.doFilter(AnonymousAuthenticationFilter.java:100)
    at 
org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:374)
    at 
org.springframework.security.oauth2.server.resource.web.authentication.BearerTokenAuthenticationFilter.doFilterInternal(BearerTokenAuthenticationFilter.java:145)
    at 
org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:116)
    at 
org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:374)
    at 

Re: [PR] NIFI-12411: Update PublishAMQP to read AMQP headers value from FlowFile attributes and `amq$headers` string [nifi]

2024-05-14 Thread via GitHub


umarhussain15 commented on PR #8105:
URL: https://github.com/apache/nifi/pull/8105#issuecomment-2110984083

   @exceptionfactory @joewitt 
   
   I have added one more change, which will allow users to set amqp processor's 
password via env variables.
   I think this will be helpful in cases where we use AMQP processors in 
multiple NiFI instances. Instead of copying and pasting different passwords in 
processor properties, we can reference the env variable via its name.
   Let me know your thoughts on it.
   Thanks.


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] NIFI-12896: Add Endpoint Override URL property for PutSNS processor [nifi]

2024-05-14 Thread via GitHub


sujkm commented on PR #8529:
URL: https://github.com/apache/nifi/pull/8529#issuecomment-2110945875

   PR has been rebased on latest main branch


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13234) Unauthorized canvas components issues

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13234:
---
Summary: Unauthorized canvas components issues  (was: Unauthorized PG 
issues)

> Unauthorized canvas components issues
> -
>
> Key: NIFI-13234
> URL: https://issues.apache.org/jira/browse/NIFI-13234
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Scott Aslan
>Priority: Major
> Attachments: Screenshot 2024-05-14 at 1.01.15 PM.png
>
>
> There are contrast ratio issues in the unauthorized PG on the canvas:
>  
> !Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13234) Unauthorized canvas components issues

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13234:
---
Description: 
There are color and contrast ratio issues for unauthorized components on the 
canvas:

 

!Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!

  was:
There are contrast ratio issues in the unauthorized PG on the canvas:

 

!Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!


> Unauthorized canvas components issues
> -
>
> Key: NIFI-13234
> URL: https://issues.apache.org/jira/browse/NIFI-13234
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Scott Aslan
>Priority: Major
> Attachments: Screenshot 2024-05-14 at 1.01.15 PM.png
>
>
> There are color and contrast ratio issues for unauthorized components on the 
> canvas:
>  
> !Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[PR] NIFI-13235 Use the PG id from the entity because component may be nul… [nifi]

2024-05-14 Thread via GitHub


bbende opened a new pull request, #8831:
URL: https://github.com/apache/nifi/pull/8831

   …l if current user does not have READ permissions
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   # Summary
   
   [NIFI-13235](https://issues.apache.org/jira/browse/NIFI-13235)
   
   # Tracking
   
   Please complete the following tracking steps prior to pull request creation.
   
   ### Issue Tracking
   
   - [X] [Apache NiFi Jira](https://issues.apache.org/jira/browse/NIFI) issue 
created
   
   ### Pull Request Tracking
   
   - [X] Pull Request title starts with Apache NiFi Jira issue number, such as 
`NIFI-0`
   - [X] Pull Request commit message starts with Apache NiFi Jira issue number, 
as such `NIFI-0`
   
   ### Pull Request Formatting
   
   - [X] Pull Request based on current revision of the `main` branch
   - [X] Pull Request refers to a feature branch with one commit containing 
changes
   
   # Verification
   
   Please indicate the verification steps performed prior to pull request 
creation.
   
   ### Build
   
   - [X] Build completed using `mvn clean install -P contrib-check`
 - [X] JDK 21
   
   ### UI Contributions
   
   - [ ] NiFi is modernizing its UI. Any contributions that update the [current 
UI](https://github.com/apache/nifi/tree/main/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui)
 also need to be implemented in the [new 
UI](https://github.com/apache/nifi/tree/main/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-frontend/src/main/nifi).
  
   
   ### Licensing
   
   - [ ] New dependencies are compatible with the [Apache License 
2.0](https://apache.org/licenses/LICENSE-2.0) according to the [License 
Policy](https://www.apache.org/legal/resolved.html)
   - [ ] New dependencies are documented in applicable `LICENSE` and `NOTICE` 
files
   
   ### Documentation
   
   - [ ] Documentation formatting appears as expected in rendered files
   


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13234) Unauthorized PG issues

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13234:
---
Summary: Unauthorized PG issues  (was: Ghosted PG issues)

> Unauthorized PG issues
> --
>
> Key: NIFI-13234
> URL: https://issues.apache.org/jira/browse/NIFI-13234
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Scott Aslan
>Priority: Major
> Attachments: Screenshot 2024-05-14 at 1.01.15 PM.png
>
>
> There are contrast ratio issues in the ghosted PG on the canvas:
>  
> !Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13234) Unauthorized PG issues

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13234:
---
Description: 
There are contrast ratio issues in the unauthorized PG on the canvas:

 

!Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!

  was:
There are contrast ratio issues in the ghosted PG on the canvas:

 

!Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!


> Unauthorized PG issues
> --
>
> Key: NIFI-13234
> URL: https://issues.apache.org/jira/browse/NIFI-13234
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Scott Aslan
>Priority: Major
> Attachments: Screenshot 2024-05-14 at 1.01.15 PM.png
>
>
> There are contrast ratio issues in the unauthorized PG on the canvas:
>  
> !Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13235) Import from registry to a PG without read permissions causes NPE

2024-05-14 Thread Bryan Bende (Jira)
Bryan Bende created NIFI-13235:
--

 Summary: Import from registry to a PG without read permissions 
causes NPE
 Key: NIFI-13235
 URL: https://issues.apache.org/jira/browse/NIFI-13235
 Project: Apache NiFi
  Issue Type: Bug
Affects Versions: 2.0.0-M2
Reporter: Bryan Bende


If you are inside a PG where you have WRITE permissions, but not READ 
permissions, and you import a new PG from a registry client, it results in an 
unexpected error page and an NPE in the logs:
{code:java}
2024-05-14 13:41:46,946 ERROR [NiFi Web Server-139] 
o.a.nifi.web.api.config.ThrowableMapper An unexpected error has occurred: 
java.lang.NullPointerException: Cannot invoke 
"org.apache.nifi.web.api.dto.ProcessGroupDTO.getId()" because the return value 
of "org.apache.nifi.web.api.entity.ProcessGroupEntity.getComponent()" is null. 
Returning Internal Server Error response.
java.lang.NullPointerException: Cannot invoke 
"org.apache.nifi.web.api.dto.ProcessGroupDTO.getId()" because the return value 
of "org.apache.nifi.web.api.entity.ProcessGroupEntity.getComponent()" is null
        at 
org.apache.nifi.web.api.ProcessGroupResource.lambda$createProcessGroup$18(ProcessGroupResource.java:1016)
        at 
org.apache.nifi.web.api.ApplicationResource.withWriteLock(ApplicationResource.java:691)
        at 
org.apache.nifi.web.api.ProcessGroupResource.createProcessGroup(ProcessGroupResource.java:988)
 {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13234) Ghosted PG issues

2024-05-14 Thread Scott Aslan (Jira)
Scott Aslan created NIFI-13234:
--

 Summary: Ghosted PG issues
 Key: NIFI-13234
 URL: https://issues.apache.org/jira/browse/NIFI-13234
 Project: Apache NiFi
  Issue Type: Sub-task
Reporter: Scott Aslan
 Attachments: Screenshot 2024-05-14 at 1.01.15 PM.png

There are contrast ratio issues in the ghosted PG on the canvas:

 

!Screenshot 2024-05-14 at 1.01.15 PM.png|width=415,height=222!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13233) Change version dialog issues

2024-05-14 Thread Scott Aslan (Jira)
Scott Aslan created NIFI-13233:
--

 Summary: Change version dialog issues
 Key: NIFI-13233
 URL: https://issues.apache.org/jira/browse/NIFI-13233
 Project: Apache NiFi
  Issue Type: Sub-task
Reporter: Scott Aslan
 Attachments: Screenshot 2024-05-14 at 1.21.56 PM.png

There are some text overflow issues and the version column does not have enough 
width for the new git backed version name.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Resolved] (NIFI-13229) Submit branch, when appropriate, to backend when starting version control

2024-05-14 Thread Bryan Bende (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13229?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bryan Bende resolved NIFI-13229.

Fix Version/s: 2.0.0-M4
   Resolution: Fixed

> Submit branch, when appropriate, to backend when starting version control
> -
>
> Key: NIFI-13229
> URL: https://issues.apache.org/jira/browse/NIFI-13229
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Rob Fellows
>Assignee: Rob Fellows
>Priority: Major
> Fix For: 2.0.0-M4
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-13229) Submit branch, when appropriate, to backend when starting version control

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-13229?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846379#comment-17846379
 ] 

ASF subversion and git services commented on NIFI-13229:


Commit d20fa9417dfaddb7394be017256735ae1ef17c02 in nifi's branch 
refs/heads/main from Rob Fellows
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=d20fa9417d ]

[NIFI-13229] - Include branch when submitting to save version control endpoint. 
(#8830)



> Submit branch, when appropriate, to backend when starting version control
> -
>
> Key: NIFI-13229
> URL: https://issues.apache.org/jira/browse/NIFI-13229
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Rob Fellows
>Assignee: Rob Fellows
>Priority: Major
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] [NIFI-13229] - Include branch when submitting to save version control endpoint [nifi]

2024-05-14 Thread via GitHub


bbende merged PR #8830:
URL: https://github.com/apache/nifi/pull/8830


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [NIFI-13082] Created SplitPcap processor, Pcap supporting class, and … [nifi]

2024-05-14 Thread via GitHub


dan-s1 commented on code in PR #8691:
URL: https://github.com/apache/nifi/pull/8691#discussion_r1600325204


##
nifi-extension-bundles/nifi-network-bundle/nifi-network-processors/src/main/java/org/apache/nifi/processors/network/SplitPCAP.java:
##
@@ -0,0 +1,203 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.network;
+
+import org.apache.nifi.components.PropertyDescriptor;
+import org.apache.nifi.flowfile.FlowFile;
+import org.apache.nifi.annotation.lifecycle.OnScheduled;
+import org.apache.nifi.annotation.behavior.WritesAttribute;
+import org.apache.nifi.annotation.documentation.CapabilityDescription;
+import org.apache.nifi.annotation.documentation.Tags;
+import org.apache.nifi.processor.AbstractProcessor;
+import org.apache.nifi.processor.ProcessContext;
+import org.apache.nifi.processor.ProcessSession;
+import org.apache.nifi.processor.Relationship;
+import org.apache.nifi.processor.util.StandardValidators;
+import org.apache.nifi.processors.network.util.PCAP;
+import org.apache.nifi.processors.network.util.PCAP.ByteBufferInterface;
+import org.apache.nifi.processors.network.util.PCAP.Packet;
+
+import java.io.ByteArrayOutputStream;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+@Tags({"PCAP", "Splitter", "Network", "Packet", "Capture", "Wireshark", 
"TShark"})
+@CapabilityDescription("Splits a pcap file into multiple pcap files based on a 
maximum size.")
+@WritesAttribute(attribute = "ERROR_REASON", description = "The reason the 
flowfile was sent to the failure relationship.")
+
+public class SplitPCAP extends AbstractProcessor {
+

Review Comment:
   Sorry about that. I am not sure `public` is needed. I believe package 
visibility would work also.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [NIFI-13082] Created SplitPcap processor, Pcap supporting class, and … [nifi]

2024-05-14 Thread via GitHub


dan-s1 commented on code in PR #8691:
URL: https://github.com/apache/nifi/pull/8691#discussion_r1600325204


##
nifi-extension-bundles/nifi-network-bundle/nifi-network-processors/src/main/java/org/apache/nifi/processors/network/SplitPCAP.java:
##
@@ -0,0 +1,203 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.network;
+
+import org.apache.nifi.components.PropertyDescriptor;
+import org.apache.nifi.flowfile.FlowFile;
+import org.apache.nifi.annotation.lifecycle.OnScheduled;
+import org.apache.nifi.annotation.behavior.WritesAttribute;
+import org.apache.nifi.annotation.documentation.CapabilityDescription;
+import org.apache.nifi.annotation.documentation.Tags;
+import org.apache.nifi.processor.AbstractProcessor;
+import org.apache.nifi.processor.ProcessContext;
+import org.apache.nifi.processor.ProcessSession;
+import org.apache.nifi.processor.Relationship;
+import org.apache.nifi.processor.util.StandardValidators;
+import org.apache.nifi.processors.network.util.PCAP;
+import org.apache.nifi.processors.network.util.PCAP.ByteBufferInterface;
+import org.apache.nifi.processors.network.util.PCAP.Packet;
+
+import java.io.ByteArrayOutputStream;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+@Tags({"PCAP", "Splitter", "Network", "Packet", "Capture", "Wireshark", 
"TShark"})
+@CapabilityDescription("Splits a pcap file into multiple pcap files based on a 
maximum size.")
+@WritesAttribute(attribute = "ERROR_REASON", description = "The reason the 
flowfile was sent to the failure relationship.")
+
+public class SplitPCAP extends AbstractProcessor {
+

Review Comment:
   Sorry about that. I am not sure `public `is needed. I believe package 
visibility would work also.



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] MINIFICPP-2358 Skip File permission tests while running as root [nifi-minifi-cpp]

2024-05-14 Thread via GitHub


martinzink opened a new pull request, #1789:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1789

   Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
   
   In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken:
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?
   
   - [ ] Does your PR title start with MINIFICPP- where  is the JIRA 
number you are trying to resolve? Pay particular attention to the hyphen "-" 
character.
   
   - [ ] Has your PR been rebased against the latest commit within the target 
branch (typically main)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   ### For code changes:
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the LICENSE file?
   - [ ] If applicable, have you updated the NOTICE file?
   
   ### For documentation related changes:
   - [ ] Have you ensured that format looks appropriate for the output in which 
it is rendered?
   
   ### Note:
   Please ensure that once the PR is submitted, you check GitHub Actions CI 
results for build issues and submit an update to your PR as soon as possible.
   


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] [NIFI-13082] Created SplitPcap processor, Pcap supporting class, and … [nifi]

2024-05-14 Thread via GitHub


JackHintonSmartDCSIT commented on code in PR #8691:
URL: https://github.com/apache/nifi/pull/8691#discussion_r1600263921


##
nifi-extension-bundles/nifi-network-bundle/nifi-network-processors/src/main/java/org/apache/nifi/processors/network/SplitPCAP.java:
##
@@ -0,0 +1,203 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.nifi.processors.network;
+
+import org.apache.nifi.components.PropertyDescriptor;
+import org.apache.nifi.flowfile.FlowFile;
+import org.apache.nifi.annotation.lifecycle.OnScheduled;
+import org.apache.nifi.annotation.behavior.WritesAttribute;
+import org.apache.nifi.annotation.documentation.CapabilityDescription;
+import org.apache.nifi.annotation.documentation.Tags;
+import org.apache.nifi.processor.AbstractProcessor;
+import org.apache.nifi.processor.ProcessContext;
+import org.apache.nifi.processor.ProcessSession;
+import org.apache.nifi.processor.Relationship;
+import org.apache.nifi.processor.util.StandardValidators;
+import org.apache.nifi.processors.network.util.PCAP;
+import org.apache.nifi.processors.network.util.PCAP.ByteBufferInterface;
+import org.apache.nifi.processors.network.util.PCAP.Packet;
+
+import java.io.ByteArrayOutputStream;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+@Tags({"PCAP", "Splitter", "Network", "Packet", "Capture", "Wireshark", 
"TShark"})
+@CapabilityDescription("Splits a pcap file into multiple pcap files based on a 
maximum size.")
+@WritesAttribute(attribute = "ERROR_REASON", description = "The reason the 
flowfile was sent to the failure relationship.")
+
+public class SplitPCAP extends AbstractProcessor {
+

Review Comment:
   I had to make this public or it wasn't visible from line 41



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-12669) EvaluateXQuery processor incorrectly encodes result attributes

2024-05-14 Thread Matt Burgess (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Burgess updated NIFI-12669:

Fix Version/s: 2.0.0-M4
   (was: 2.0.0)

> EvaluateXQuery processor incorrectly encodes result attributes
> --
>
> Key: NIFI-12669
> URL: https://issues.apache.org/jira/browse/NIFI-12669
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Configuration, Extensions
> Environment: JVM with non-UTF-8 default encoding (e.g. default 
> Windows installation)
>Reporter: René Zeidler
>Assignee: Jim Steinebrey
>Priority: Major
>  Labels: encoding, utf8, windows, xml
> Fix For: 1.27.0, 2.0.0-M4
>
> Attachments: EvaluateXQuery_Encoding_Bug.json, 
> image-2024-01-25-10-24-17-005.png, image-2024-01-25-10-31-35-200.png
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> h2. Environment
> This issue affects environments where the JVM default encoding is not 
> {{{}UTF-8{}}}. Standard Java installations on Windows are affected, as they 
> usually use the default encoding {{{}windows-1252{}}}. To reproduce the issue 
> on Linux, change the default encoding to {{windows-1252}} by adding the 
> following line to your {{{}bootstrap.conf{}}}:
> {quote}{{java.arg.21=-Dfile.encoding=windows-1252}}
> {quote}
> h2. Summary
> The EvaluateXQuery incorrectly encodes result values when storing them in 
> attributes. This causes non-ASCII characters to be garbled.
> Example:
> !image-2024-01-25-10-24-17-005.png!
> h2. Steps to reproduce
>  # Make sure NiFi runs with a non-UTF-8 default encoding, see "Environment"
>  # Create a GenerateFlowFile processor with the following content:
> {quote}{{}}
> {{}}
> {{  This text contains non-ASCII characters: ÄÖÜäöüßéèóò}}
> {{}}
> {quote}
>  # Connect the processor to an EvaluateXQuery processor.
> Set the {{Destination}} to {{{}flowfile-attribute{}}}.
> Create a custom property {{myData}} with value {{{}string(/myRoot/myData){}}}.
>  # Connect the outputs of the EvaluateXQuery processor to funnels to be able 
> to observe the result in the queue.
>  # Start the EvaluateXQuery processor and run the GenerateFlowFile processor 
> once.
> The flow should look similar to this:
> !image-2024-01-25-10-31-35-200.png!
> I also attached a JSON export of the example flow.
>  # Observe the attributes of the resulting FlowFile in the queue.
> h3. Expected Result
> The FlowFile should contain an attribute {{myData}} with the value {{{}"This 
> text contains non-ASCII characters: ÄÖÜäöüßéèóò"{}}}.
> h3. Actual Result
> The attribute has the value "This text contains non-ASCII characters: 
> ÄÖÜäöüßéèóò".
> h2. Root Cause Analysis
> EvaluateXQuery uses the method 
> [{{formatItem}}|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L368-L372]
>  to write the query result to an attribute. This method calls 
> {{{}ByteArrayOutputStream{}}}'s 
> [toString|https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/ByteArrayOutputStream.html#toString()]
>  method without an encoding argument, which then defaults to the default 
> charset of the environment. Bytes are always written to this output stream 
> using UTF-8 
> ([.getBytes(StandardCharsets.UTF8)|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L397]).
>  When the default charset is not UTF-8, this results in UTF-8 bytes to be 
> interpreted in a different encoding when converting to a string, resulting in 
> garbled text (see above).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] MINIFICPP-2346: Build MiNiFi Core, MainExe, Standard-Processors with Conan2 [nifi-minifi-cpp]

2024-05-14 Thread via GitHub


james94 commented on PR #1775:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1775#issuecomment-2110550068

   @martinzink thanks for the suggestion. I brought MINIFI-2346 to up to date 
with latest upstream MiNiFi C++ main branch. Ubuntu 20.04 and Docker build for 
integration tests passed. The other CI/CDs failed. Its probably related to some 
of the changes I am introducing to the build infrastructure. I have mainly been 
testing the updates in a Ubuntu 22.04 docker container. I haven't tested them 
yet on Windows or Mac OS X. Those 2 last OSs, I'd test later. I also would need 
to look into why CI/CD is failing too. After I finish refactoring build 
infrastructure to support Conan version 2 and keep backward compatibility with 
the standalone CMake approach, I will work toward updating the MiNiFi bootstrap 
py script with conan support too. I will get to the CI/CD issues as soon as I 
can, but those will probably be the last updates.


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] MINIFICPP-2338 Move out ExecuteProcess from standard processors [nifi-minifi-cpp]

2024-05-14 Thread via GitHub


martinzink opened a new pull request, #1788:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1788

   Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
   
   In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken:
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?
   
   - [ ] Does your PR title start with MINIFICPP- where  is the JIRA 
number you are trying to resolve? Pay particular attention to the hyphen "-" 
character.
   
   - [ ] Has your PR been rebased against the latest commit within the target 
branch (typically main)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   ### For code changes:
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the LICENSE file?
   - [ ] If applicable, have you updated the NOTICE file?
   
   ### For documentation related changes:
   - [ ] Have you ensured that format looks appropriate for the output in which 
it is rendered?
   
   ### Note:
   Please ensure that once the PR is submitted, you check GitHub Actions CI 
results for build issues and submit an update to your PR as soon as possible.
   


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (NIFI-13232) Add REST API Resource for Authentication Configuration

2024-05-14 Thread David Handermann (Jira)
David Handermann created NIFI-13232:
---

 Summary: Add REST API Resource for Authentication Configuration
 Key: NIFI-13232
 URL: https://issues.apache.org/jira/browse/NIFI-13232
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Core Framework
Reporter: David Handermann
Assignee: David Handermann


The NiFi user interface applications include Servlet Filters to handle Login 
and Logout redirection based on the configured authentication strategy.

Rather than continuing to maintain these filters, a new REST API resource 
should be created to provide login and logout URLs based on application 
configuration. The response should also indicate whether the system supports 
login and logout operations so that the user interface can present the 
appropriate forms and links according to the current system configuration.

Implementing this new REST API should support the removal of the Login and 
Logout filters from the new user interface, simplifying the build and 
deployment process.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12669) EvaluateXQuery processor incorrectly encodes result attributes

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12669:

Resolution: Fixed
Status: Resolved  (was: Patch Available)

> EvaluateXQuery processor incorrectly encodes result attributes
> --
>
> Key: NIFI-12669
> URL: https://issues.apache.org/jira/browse/NIFI-12669
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Configuration, Extensions
> Environment: JVM with non-UTF-8 default encoding (e.g. default 
> Windows installation)
>Reporter: René Zeidler
>Assignee: Jim Steinebrey
>Priority: Major
>  Labels: encoding, utf8, windows, xml
> Fix For: 1.27.0, 2.0.0
>
> Attachments: EvaluateXQuery_Encoding_Bug.json, 
> image-2024-01-25-10-24-17-005.png, image-2024-01-25-10-31-35-200.png
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> h2. Environment
> This issue affects environments where the JVM default encoding is not 
> {{{}UTF-8{}}}. Standard Java installations on Windows are affected, as they 
> usually use the default encoding {{{}windows-1252{}}}. To reproduce the issue 
> on Linux, change the default encoding to {{windows-1252}} by adding the 
> following line to your {{{}bootstrap.conf{}}}:
> {quote}{{java.arg.21=-Dfile.encoding=windows-1252}}
> {quote}
> h2. Summary
> The EvaluateXQuery incorrectly encodes result values when storing them in 
> attributes. This causes non-ASCII characters to be garbled.
> Example:
> !image-2024-01-25-10-24-17-005.png!
> h2. Steps to reproduce
>  # Make sure NiFi runs with a non-UTF-8 default encoding, see "Environment"
>  # Create a GenerateFlowFile processor with the following content:
> {quote}{{}}
> {{}}
> {{  This text contains non-ASCII characters: ÄÖÜäöüßéèóò}}
> {{}}
> {quote}
>  # Connect the processor to an EvaluateXQuery processor.
> Set the {{Destination}} to {{{}flowfile-attribute{}}}.
> Create a custom property {{myData}} with value {{{}string(/myRoot/myData){}}}.
>  # Connect the outputs of the EvaluateXQuery processor to funnels to be able 
> to observe the result in the queue.
>  # Start the EvaluateXQuery processor and run the GenerateFlowFile processor 
> once.
> The flow should look similar to this:
> !image-2024-01-25-10-31-35-200.png!
> I also attached a JSON export of the example flow.
>  # Observe the attributes of the resulting FlowFile in the queue.
> h3. Expected Result
> The FlowFile should contain an attribute {{myData}} with the value {{{}"This 
> text contains non-ASCII characters: ÄÖÜäöüßéèóò"{}}}.
> h3. Actual Result
> The attribute has the value "This text contains non-ASCII characters: 
> ÄÖÜäöüßéèóò".
> h2. Root Cause Analysis
> EvaluateXQuery uses the method 
> [{{formatItem}}|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L368-L372]
>  to write the query result to an attribute. This method calls 
> {{{}ByteArrayOutputStream{}}}'s 
> [toString|https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/ByteArrayOutputStream.html#toString()]
>  method without an encoding argument, which then defaults to the default 
> charset of the environment. Bytes are always written to this output stream 
> using UTF-8 
> ([.getBytes(StandardCharsets.UTF8)|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L397]).
>  When the default charset is not UTF-8, this results in UTF-8 bytes to be 
> interpreted in a different encoding when converting to a string, resulting in 
> garbled text (see above).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12669) EvaluateXQuery processor incorrectly encodes result attributes

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12669:

Fix Version/s: 1.27.0
   2.0.0

> EvaluateXQuery processor incorrectly encodes result attributes
> --
>
> Key: NIFI-12669
> URL: https://issues.apache.org/jira/browse/NIFI-12669
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Configuration, Extensions
> Environment: JVM with non-UTF-8 default encoding (e.g. default 
> Windows installation)
>Reporter: René Zeidler
>Assignee: Jim Steinebrey
>Priority: Major
>  Labels: encoding, utf8, windows, xml
> Fix For: 1.27.0, 2.0.0
>
> Attachments: EvaluateXQuery_Encoding_Bug.json, 
> image-2024-01-25-10-24-17-005.png, image-2024-01-25-10-31-35-200.png
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> h2. Environment
> This issue affects environments where the JVM default encoding is not 
> {{{}UTF-8{}}}. Standard Java installations on Windows are affected, as they 
> usually use the default encoding {{{}windows-1252{}}}. To reproduce the issue 
> on Linux, change the default encoding to {{windows-1252}} by adding the 
> following line to your {{{}bootstrap.conf{}}}:
> {quote}{{java.arg.21=-Dfile.encoding=windows-1252}}
> {quote}
> h2. Summary
> The EvaluateXQuery incorrectly encodes result values when storing them in 
> attributes. This causes non-ASCII characters to be garbled.
> Example:
> !image-2024-01-25-10-24-17-005.png!
> h2. Steps to reproduce
>  # Make sure NiFi runs with a non-UTF-8 default encoding, see "Environment"
>  # Create a GenerateFlowFile processor with the following content:
> {quote}{{}}
> {{}}
> {{  This text contains non-ASCII characters: ÄÖÜäöüßéèóò}}
> {{}}
> {quote}
>  # Connect the processor to an EvaluateXQuery processor.
> Set the {{Destination}} to {{{}flowfile-attribute{}}}.
> Create a custom property {{myData}} with value {{{}string(/myRoot/myData){}}}.
>  # Connect the outputs of the EvaluateXQuery processor to funnels to be able 
> to observe the result in the queue.
>  # Start the EvaluateXQuery processor and run the GenerateFlowFile processor 
> once.
> The flow should look similar to this:
> !image-2024-01-25-10-31-35-200.png!
> I also attached a JSON export of the example flow.
>  # Observe the attributes of the resulting FlowFile in the queue.
> h3. Expected Result
> The FlowFile should contain an attribute {{myData}} with the value {{{}"This 
> text contains non-ASCII characters: ÄÖÜäöüßéèóò"{}}}.
> h3. Actual Result
> The attribute has the value "This text contains non-ASCII characters: 
> ÄÖÜäöüßéèóò".
> h2. Root Cause Analysis
> EvaluateXQuery uses the method 
> [{{formatItem}}|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L368-L372]
>  to write the query result to an attribute. This method calls 
> {{{}ByteArrayOutputStream{}}}'s 
> [toString|https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/ByteArrayOutputStream.html#toString()]
>  method without an encoding argument, which then defaults to the default 
> charset of the environment. Bytes are always written to this output stream 
> using UTF-8 
> ([.getBytes(StandardCharsets.UTF8)|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L397]).
>  When the default charset is not UTF-8, this results in UTF-8 bytes to be 
> interpreted in a different encoding when converting to a string, resulting in 
> garbled text (see above).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13231) Update GitHubFlowRegistryClient to generate App Installation Token

2024-05-14 Thread Brian Ghigiarelli (Jira)
Brian Ghigiarelli created NIFI-13231:


 Summary: Update GitHubFlowRegistryClient to generate App 
Installation Token
 Key: NIFI-13231
 URL: https://issues.apache.org/jira/browse/NIFI-13231
 Project: Apache NiFi
  Issue Type: Improvement
Affects Versions: 2.0.0-M3
Reporter: Brian Ghigiarelli


The new GitHubFlowRegistryClient accepts two types of authentication:
 # Personal Access Token
 # App Installation Token

The App Installation Token requires the user to input the token as a property, 
then uses that to communicate with GitHub. However, this token is short-lived 
and would require frequent updates (~hourly).

Instead of directly providing the token, the GitHubFlowRegistryClient should 
accept as properties:
 # App ID
 # Private Key (PEM text format)

and use these values to generate a short-lived token, following the guide at 
https://docs.github.com/en/apps/creating-github-apps/authenticating-with-a-github-app/generating-a-json-web-token-jwt-for-a-github-app#generating-a-json-web-token-jwt



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13230) Include Flow Registry Clients in Docs

2024-05-14 Thread Brian Ghigiarelli (Jira)
Brian Ghigiarelli created NIFI-13230:


 Summary: Include Flow Registry Clients in Docs
 Key: NIFI-13230
 URL: https://issues.apache.org/jira/browse/NIFI-13230
 Project: Apache NiFi
  Issue Type: Improvement
  Components: Documentation  Website
Affects Versions: 2.0.0-M2, 2.0.0-M3
Reporter: Brian Ghigiarelli


Flow Registry Clients are not yet included in the generated help docs. There is 
only a section in the User Guide for how to configure a registry client: 
https://nifi.apache.org/documentation/nifi-2.0.0-M2/html/user-guide.html#connecting-to-a-nifi-registry

Two types exist now (one for NiFi Registry and one for GitHub), but there is 
room to expand to other registries.

Flow Registry Clients can be included in a new section within the docs at the 
same level as Processors, Flow Analysis Rules, Reporting Tasks, etc.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-12669) EvaluateXQuery processor incorrectly encodes result attributes

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-12669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846342#comment-17846342
 ] 

ASF subversion and git services commented on NIFI-12669:


Commit e6a90eb1e8609eae2ef22f2490327451186bbe7b in nifi's branch 
refs/heads/main from Jim Steinebrey
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=e6a90eb1e8 ]

NIFI-12669 Fix EvaluateXQuery processor which incorrectly encodes result 
attributes in certain case

Signed-off-by: Matt Burgess 

This closes #8826


> EvaluateXQuery processor incorrectly encodes result attributes
> --
>
> Key: NIFI-12669
> URL: https://issues.apache.org/jira/browse/NIFI-12669
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Configuration, Extensions
> Environment: JVM with non-UTF-8 default encoding (e.g. default 
> Windows installation)
>Reporter: René Zeidler
>Assignee: Jim Steinebrey
>Priority: Major
>  Labels: encoding, utf8, windows, xml
> Attachments: EvaluateXQuery_Encoding_Bug.json, 
> image-2024-01-25-10-24-17-005.png, image-2024-01-25-10-31-35-200.png
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> h2. Environment
> This issue affects environments where the JVM default encoding is not 
> {{{}UTF-8{}}}. Standard Java installations on Windows are affected, as they 
> usually use the default encoding {{{}windows-1252{}}}. To reproduce the issue 
> on Linux, change the default encoding to {{windows-1252}} by adding the 
> following line to your {{{}bootstrap.conf{}}}:
> {quote}{{java.arg.21=-Dfile.encoding=windows-1252}}
> {quote}
> h2. Summary
> The EvaluateXQuery incorrectly encodes result values when storing them in 
> attributes. This causes non-ASCII characters to be garbled.
> Example:
> !image-2024-01-25-10-24-17-005.png!
> h2. Steps to reproduce
>  # Make sure NiFi runs with a non-UTF-8 default encoding, see "Environment"
>  # Create a GenerateFlowFile processor with the following content:
> {quote}{{}}
> {{}}
> {{  This text contains non-ASCII characters: ÄÖÜäöüßéèóò}}
> {{}}
> {quote}
>  # Connect the processor to an EvaluateXQuery processor.
> Set the {{Destination}} to {{{}flowfile-attribute{}}}.
> Create a custom property {{myData}} with value {{{}string(/myRoot/myData){}}}.
>  # Connect the outputs of the EvaluateXQuery processor to funnels to be able 
> to observe the result in the queue.
>  # Start the EvaluateXQuery processor and run the GenerateFlowFile processor 
> once.
> The flow should look similar to this:
> !image-2024-01-25-10-31-35-200.png!
> I also attached a JSON export of the example flow.
>  # Observe the attributes of the resulting FlowFile in the queue.
> h3. Expected Result
> The FlowFile should contain an attribute {{myData}} with the value {{{}"This 
> text contains non-ASCII characters: ÄÖÜäöüßéèóò"{}}}.
> h3. Actual Result
> The attribute has the value "This text contains non-ASCII characters: 
> ÄÖÜäöüßéèóò".
> h2. Root Cause Analysis
> EvaluateXQuery uses the method 
> [{{formatItem}}|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L368-L372]
>  to write the query result to an attribute. This method calls 
> {{{}ByteArrayOutputStream{}}}'s 
> [toString|https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/ByteArrayOutputStream.html#toString()]
>  method without an encoding argument, which then defaults to the default 
> charset of the environment. Bytes are always written to this output stream 
> using UTF-8 
> ([.getBytes(StandardCharsets.UTF8)|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L397]).
>  When the default charset is not UTF-8, this results in UTF-8 bytes to be 
> interpreted in a different encoding when converting to a string, resulting in 
> garbled text (see above).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-12669 Fix EvaluateXQuery processor to encode result attr correctly on Windows [nifi]

2024-05-14 Thread via GitHub


mattyb149 commented on PR #8826:
URL: https://github.com/apache/nifi/pull/8826#issuecomment-2110490625

   +1 LGTM, thanks for the fix! Merging to main and support/nifi-1.x


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] NIFI-12669 Fix EvaluateXQuery processor to encode result attr correctly on Windows [nifi]

2024-05-14 Thread via GitHub


mattyb149 closed pull request #8826: NIFI-12669 Fix EvaluateXQuery processor to 
encode result attr correctly on Windows
URL: https://github.com/apache/nifi/pull/8826


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (NIFI-12669) EvaluateXQuery processor incorrectly encodes result attributes

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-12669?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846339#comment-17846339
 ] 

ASF subversion and git services commented on NIFI-12669:


Commit 47101f760e60f2a93e524c2adb01ebe29ebb754d in nifi's branch 
refs/heads/support/nifi-1.x from Jim Steinebrey
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=47101f760e ]

NIFI-12669 Fix EvaluateXQuery processor which incorrectly encodes result 
attributes in certain case

Signed-off-by: Matt Burgess 


> EvaluateXQuery processor incorrectly encodes result attributes
> --
>
> Key: NIFI-12669
> URL: https://issues.apache.org/jira/browse/NIFI-12669
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Configuration, Extensions
> Environment: JVM with non-UTF-8 default encoding (e.g. default 
> Windows installation)
>Reporter: René Zeidler
>Assignee: Jim Steinebrey
>Priority: Major
>  Labels: encoding, utf8, windows, xml
> Attachments: EvaluateXQuery_Encoding_Bug.json, 
> image-2024-01-25-10-24-17-005.png, image-2024-01-25-10-31-35-200.png
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> h2. Environment
> This issue affects environments where the JVM default encoding is not 
> {{{}UTF-8{}}}. Standard Java installations on Windows are affected, as they 
> usually use the default encoding {{{}windows-1252{}}}. To reproduce the issue 
> on Linux, change the default encoding to {{windows-1252}} by adding the 
> following line to your {{{}bootstrap.conf{}}}:
> {quote}{{java.arg.21=-Dfile.encoding=windows-1252}}
> {quote}
> h2. Summary
> The EvaluateXQuery incorrectly encodes result values when storing them in 
> attributes. This causes non-ASCII characters to be garbled.
> Example:
> !image-2024-01-25-10-24-17-005.png!
> h2. Steps to reproduce
>  # Make sure NiFi runs with a non-UTF-8 default encoding, see "Environment"
>  # Create a GenerateFlowFile processor with the following content:
> {quote}{{}}
> {{}}
> {{  This text contains non-ASCII characters: ÄÖÜäöüßéèóò}}
> {{}}
> {quote}
>  # Connect the processor to an EvaluateXQuery processor.
> Set the {{Destination}} to {{{}flowfile-attribute{}}}.
> Create a custom property {{myData}} with value {{{}string(/myRoot/myData){}}}.
>  # Connect the outputs of the EvaluateXQuery processor to funnels to be able 
> to observe the result in the queue.
>  # Start the EvaluateXQuery processor and run the GenerateFlowFile processor 
> once.
> The flow should look similar to this:
> !image-2024-01-25-10-31-35-200.png!
> I also attached a JSON export of the example flow.
>  # Observe the attributes of the resulting FlowFile in the queue.
> h3. Expected Result
> The FlowFile should contain an attribute {{myData}} with the value {{{}"This 
> text contains non-ASCII characters: ÄÖÜäöüßéèóò"{}}}.
> h3. Actual Result
> The attribute has the value "This text contains non-ASCII characters: 
> ÄÖÜäöüßéèóò".
> h2. Root Cause Analysis
> EvaluateXQuery uses the method 
> [{{formatItem}}|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L368-L372]
>  to write the query result to an attribute. This method calls 
> {{{}ByteArrayOutputStream{}}}'s 
> [toString|https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/ByteArrayOutputStream.html#toString()]
>  method without an encoding argument, which then defaults to the default 
> charset of the environment. Bytes are always written to this output stream 
> using UTF-8 
> ([.getBytes(StandardCharsets.UTF8)|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L397]).
>  When the default charset is not UTF-8, this results in UTF-8 bytes to be 
> interpreted in a different encoding when converting to a string, resulting in 
> garbled text (see above).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12669) EvaluateXQuery processor incorrectly encodes result attributes

2024-05-14 Thread Matt Burgess (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Burgess updated NIFI-12669:

Affects Version/s: (was: 2.0.0-M1)
   (was: 1.24.0)
   (was: 1.25.0)
   (was: 2.0.0-M2)
   Status: Patch Available  (was: Open)

> EvaluateXQuery processor incorrectly encodes result attributes
> --
>
> Key: NIFI-12669
> URL: https://issues.apache.org/jira/browse/NIFI-12669
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Configuration, Extensions
> Environment: JVM with non-UTF-8 default encoding (e.g. default 
> Windows installation)
>Reporter: René Zeidler
>Assignee: Jim Steinebrey
>Priority: Major
>  Labels: encoding, utf8, windows, xml
> Attachments: EvaluateXQuery_Encoding_Bug.json, 
> image-2024-01-25-10-24-17-005.png, image-2024-01-25-10-31-35-200.png
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> h2. Environment
> This issue affects environments where the JVM default encoding is not 
> {{{}UTF-8{}}}. Standard Java installations on Windows are affected, as they 
> usually use the default encoding {{{}windows-1252{}}}. To reproduce the issue 
> on Linux, change the default encoding to {{windows-1252}} by adding the 
> following line to your {{{}bootstrap.conf{}}}:
> {quote}{{java.arg.21=-Dfile.encoding=windows-1252}}
> {quote}
> h2. Summary
> The EvaluateXQuery incorrectly encodes result values when storing them in 
> attributes. This causes non-ASCII characters to be garbled.
> Example:
> !image-2024-01-25-10-24-17-005.png!
> h2. Steps to reproduce
>  # Make sure NiFi runs with a non-UTF-8 default encoding, see "Environment"
>  # Create a GenerateFlowFile processor with the following content:
> {quote}{{}}
> {{}}
> {{  This text contains non-ASCII characters: ÄÖÜäöüßéèóò}}
> {{}}
> {quote}
>  # Connect the processor to an EvaluateXQuery processor.
> Set the {{Destination}} to {{{}flowfile-attribute{}}}.
> Create a custom property {{myData}} with value {{{}string(/myRoot/myData){}}}.
>  # Connect the outputs of the EvaluateXQuery processor to funnels to be able 
> to observe the result in the queue.
>  # Start the EvaluateXQuery processor and run the GenerateFlowFile processor 
> once.
> The flow should look similar to this:
> !image-2024-01-25-10-31-35-200.png!
> I also attached a JSON export of the example flow.
>  # Observe the attributes of the resulting FlowFile in the queue.
> h3. Expected Result
> The FlowFile should contain an attribute {{myData}} with the value {{{}"This 
> text contains non-ASCII characters: ÄÖÜäöüßéèóò"{}}}.
> h3. Actual Result
> The attribute has the value "This text contains non-ASCII characters: 
> ÄÖÜäöüßéèóò".
> h2. Root Cause Analysis
> EvaluateXQuery uses the method 
> [{{formatItem}}|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L368-L372]
>  to write the query result to an attribute. This method calls 
> {{{}ByteArrayOutputStream{}}}'s 
> [toString|https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/io/ByteArrayOutputStream.html#toString()]
>  method without an encoding argument, which then defaults to the default 
> charset of the environment. Bytes are always written to this output stream 
> using UTF-8 
> ([.getBytes(StandardCharsets.UTF8)|https://github.com/apache/nifi/blob/2e3f83eb54cbc040b5a1da5bce9a74a558f08ea4/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EvaluateXQuery.java#L397]).
>  When the default charset is not UTF-8, this results in UTF-8 bytes to be 
> interpreted in a different encoding when converting to a string, resulting in 
> garbled text (see above).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (NIFI-13229) Submit branch, when appropriate, to backend when starting version control

2024-05-14 Thread Rob Fellows (Jira)
Rob Fellows created NIFI-13229:
--

 Summary: Submit branch, when appropriate, to backend when starting 
version control
 Key: NIFI-13229
 URL: https://issues.apache.org/jira/browse/NIFI-13229
 Project: Apache NiFi
  Issue Type: Sub-task
  Components: Core UI
Reporter: Rob Fellows
Assignee: Rob Fellows






--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-13221) Display hotkey combos in context menu

2024-05-14 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-13221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846321#comment-17846321
 ] 

ASF subversion and git services commented on NIFI-13221:


Commit 7aaa5b42247e9ded4fa8d43186713b5f15717867 in nifi's branch 
refs/heads/main from Rob Fellows
[ https://gitbox.apache.org/repos/asf?p=nifi.git;h=7aaa5b4224 ]

[NIFI-13221] - display supported shortcut/hotkey combos in context menu (#8821)

This closes #8821

> Display hotkey combos in context menu
> -
>
> Key: NIFI-13221
> URL: https://issues.apache.org/jira/browse/NIFI-13221
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Rob Fellows
>Assignee: Rob Fellows
>Priority: Major
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> When appropriate, display the hotkey combinations in the context menu to 
> bring awareness to them.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13221) Display hotkey combos in context menu

2024-05-14 Thread Scott Aslan (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13221?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Aslan updated NIFI-13221:
---
Resolution: Fixed
Status: Resolved  (was: Patch Available)

> Display hotkey combos in context menu
> -
>
> Key: NIFI-13221
> URL: https://issues.apache.org/jira/browse/NIFI-13221
> Project: Apache NiFi
>  Issue Type: Sub-task
>Reporter: Rob Fellows
>Assignee: Rob Fellows
>Priority: Major
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> When appropriate, display the hotkey combinations in the context menu to 
> bring awareness to them.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] [NIFI-13221] - display supported shortcut/hotkey combos in context menu [nifi]

2024-05-14 Thread via GitHub


scottyaslan merged PR #8821:
URL: https://github.com/apache/nifi/pull/8821


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13144) Improve dependency-update detection configuration and bump various detected dependency versions

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13144:

Issue Type: Improvement  (was: Task)

> Improve dependency-update detection configuration and bump various detected 
> dependency versions
> ---
>
> Key: NIFI-13144
> URL: https://issues.apache.org/jira/browse/NIFI-13144
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> {noformat}
> com.puppycrawl.tools:checkstyle ... 10.15.0 -> 10.16.0
> software.amazon.awssdk:xray ... 2.25.40 -> 2.25.45
> com.amazonaws:aws-java-sdk-keyspaces  1.12.710 -> 1.12.715
> com.azure:azure-sdk-bom  1.2.21 -> 1.2.23
> com.google.cloud:libraries-bom ... 26.37.0 -> 26.38.0
> net.bytebuddy:byte-buddy .. 1.14.12 -> 1.14.14
> org.checkerframework:checker-qual ... 3.37.0 -> 3.43.0
> org.eclipse.jdt:ecj . 3.33.0 -> 3.37.0
> com.microsoft.azure:msal4j .. 1.14.3 -> 1.15.0
> com.google.guava:guava .. 33.1.0-jre -> 33.2.0-jre
> com.ibm.icu:icu4j ... 74.2 -> 75.1
> org.opensaml:opensaml-*  4.3.1 -> 4.3.2
> org.apache.sshd:sshd-* ... 2.12.0 -> 2.12.1
> com.nimbusds:oauth2-oidc-sdk ... 9.43.3 -> 9.43.4
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12914) Upgrade follow-redirects library to 1.15.6

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12914?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12914:

Issue Type: Improvement  (was: Task)

> Upgrade follow-redirects library to 1.15.6
> --
>
> Key: NIFI-12914
> URL: https://issues.apache.org/jira/browse/NIFI-12914
> Project: Apache NiFi
>  Issue Type: Improvement
>  Components: Core UI
>Reporter: Matt Gilman
>Priority: Minor
> Fix For: 2.0.0-M3
>
>
> Bump follow-redirects from 1.15.5 to 1.15.6



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12853) Refactor FlowFilePrioritizer using updated Java APIs

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12853?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12853:

Issue Type: Improvement  (was: Task)

> Refactor FlowFilePrioritizer using updated Java APIs
> 
>
> Key: NIFI-12853
> URL: https://issues.apache.org/jira/browse/NIFI-12853
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: endzeit
>Assignee: endzeit
>Priority: Minor
> Fix For: 2.0.0-M3
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> The FlowFilePrioritizer implementations and test contain some boilerplate / 
> duplication that can be reduced by using updated Java APIs.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12914) Upgrade follow-redirects library to 1.15.6

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12914?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12914:

Priority: Minor  (was: Major)

> Upgrade follow-redirects library to 1.15.6
> --
>
> Key: NIFI-12914
> URL: https://issues.apache.org/jira/browse/NIFI-12914
> Project: Apache NiFi
>  Issue Type: Task
>  Components: Core UI
>Reporter: Matt Gilman
>Priority: Minor
> Fix For: 2.0.0-M3
>
>
> Bump follow-redirects from 1.15.5 to 1.15.6



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12914) Upgrade follow-redirects library to 1.15.6

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12914?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12914:

Summary: Upgrade follow-redirects library to 1.15.6  (was: Dependabot: 
follow-redirects)

> Upgrade follow-redirects library to 1.15.6
> --
>
> Key: NIFI-12914
> URL: https://issues.apache.org/jira/browse/NIFI-12914
> Project: Apache NiFi
>  Issue Type: Task
>  Components: Core UI
>Reporter: Matt Gilman
>Priority: Major
> Fix For: 2.0.0-M3
>
>
> Bump follow-redirects from 1.15.5 to 1.15.6



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13215) Update AWS, OpenTelemetry, latest dependency scan findings

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13215?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13215:

Priority: Minor  (was: Major)

> Update AWS, OpenTelemetry, latest dependency scan findings
> --
>
> Key: NIFI-13215
> URL: https://issues.apache.org/jira/browse/NIFI-13215
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Minor
> Fix For: 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13215) Update AWS, OpenTelemetry, latest dependency scan findings

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13215?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13215:

Issue Type: Improvement  (was: Task)

> Update AWS, OpenTelemetry, latest dependency scan findings
> --
>
> Key: NIFI-13215
> URL: https://issues.apache.org/jira/browse/NIFI-13215
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12853) Refactor FlowFilePrioritizer using updated Java APIs

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12853?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12853:

Priority: Minor  (was: Major)

> Refactor FlowFilePrioritizer using updated Java APIs
> 
>
> Key: NIFI-12853
> URL: https://issues.apache.org/jira/browse/NIFI-12853
> Project: Apache NiFi
>  Issue Type: Task
>Reporter: endzeit
>Assignee: endzeit
>Priority: Minor
> Fix For: 2.0.0-M3
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> The FlowFilePrioritizer implementations and test contain some boilerplate / 
> duplication that can be reduced by using updated Java APIs.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13113) Upgrade Iceberg from 1.5.0 to 1.5.2

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13113?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13113:

Summary: Upgrade Iceberg from 1.5.0 to 1.5.2  (was: upgrade Iceberg from 
1.5.0 to 1.5.2)

> Upgrade Iceberg from 1.5.0 to 1.5.2
> ---
>
> Key: NIFI-13113
> URL: https://issues.apache.org/jira/browse/NIFI-13113
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Sean Sullivan
>Assignee: Joe Witt
>Priority: Minor
> Fix For: 2.0.0-M3
>
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
>  
> Iceberg 1.5.2
>  
> [https://iceberg.apache.org/releases/]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13193) Update maven plugins to latest and require minimum of Apache Maven 3.9.6

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13193?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13193:

Issue Type: Improvement  (was: Task)

> Update maven plugins to latest and require minimum of Apache Maven 3.9.6
> 
>
> Key: NIFI-13193
> URL: https://issues.apache.org/jira/browse/NIFI-13193
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Apache Maven 3.8.5 is more than two years old already and many plugin updates 
> suggest they require at least that.  Bumping versions found during plugin 
> update scanning and bumping to that minimum maven version.
> {noformat}
> Update Maven Plugins
> org.codehaus.mojo:exec-maven-plugin  3.1.0 -> 3.2.0
> maven-clean-plugin . 3.3.1 -> 3.3.2
> com.github.eirslett:frontend-maven-plugin  1.14.2 -> 1.15.0
> com.coderplus.maven.plugins:copy-rename-maven-plugin . 1.0 -> 1.0.1
> org.codehaus.mojo:license-maven-plugin . 2.2.0 -> 2.4.0
> org.asciidoctor:asciidoctor-maven-plugin ... 2.2.4 -> 2.2.6 (or 3.0.0 
> if we want to require maven 3.8.5)
> com.googlecode.maven-download-plugin:download-maven-plugin  1.7.1 -> 1.9.0
> com.github.spotbugs:spotbugs-maven-plugin .. 4.7.3.5 -> 4.8.5.0
> maven-jar-plugin ... 3.3.0 -> 3.4.1
> org.codehaus.mojo:jaxb2-maven-plugin ... 3.1.0 -> 3.2.0
> org.patrodyne.jvnet:hisrc-higherjaxb40-maven-plugin  2.1.1 -> 2.2.0 (require 
> maven 3.8.4)
> org.jacoco:jacoco-maven-plugin ... 0.8.11 -> 0.8.12
> io.swagger.core.v3:swagger-maven-plugin-jakarta .. 2.2.20 -> 2.2.21
> io.swagger.codegen.v3:swagger-codegen-maven-plugin  3.0.52 -> 3.0.55
> {noformat}
> Not worth doing now as it causes some failure and we need to run that down.
> io.fabric8:docker-maven-plugin ... 0.43.4 -> 0.44.0



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13178) Upgrade Apache Mina, UnboundID Ldap, oktthp-digest, reactor-netty, jetbrains-annotations, reactor-core

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13178?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13178:

Issue Type: Improvement  (was: Task)

> Upgrade Apache Mina, UnboundID Ldap, oktthp-digest, reactor-netty, 
> jetbrains-annotations, reactor-core
> --
>
> Key: NIFI-13178
> URL: https://issues.apache.org/jira/browse/NIFI-13178
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13179) Bump oauth-oidc-sdk 9.43.4 to 11.10.1

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13179:

Issue Type: Improvement  (was: Task)

> Bump oauth-oidc-sdk 9.43.4 to 11.10.1
> -
>
> Key: NIFI-13179
> URL: https://issues.apache.org/jira/browse/NIFI-13179
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> oauth-oidc-sdk 9.43.4 (sep 22) to 11.10.1 (Mar 24)
> Seems relatively safe. 
> https://bitbucket.org/connect2id/oauth-2.0-sdk-with-openid-connect-extensions/src/master/CHANGELOG.txt.
>  Read all release notes and verified noted dependency changes are already in 
> our build anyway.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13179) Upgrade oauth-oidc-sdk 9.43.4 to 11.10.1

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13179:

Summary: Upgrade oauth-oidc-sdk 9.43.4 to 11.10.1  (was: Bump 
oauth-oidc-sdk 9.43.4 to 11.10.1)

> Upgrade oauth-oidc-sdk 9.43.4 to 11.10.1
> 
>
> Key: NIFI-13179
> URL: https://issues.apache.org/jira/browse/NIFI-13179
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> oauth-oidc-sdk 9.43.4 (sep 22) to 11.10.1 (Mar 24)
> Seems relatively safe. 
> https://bitbucket.org/connect2id/oauth-2.0-sdk-with-openid-connect-extensions/src/master/CHANGELOG.txt.
>  Read all release notes and verified noted dependency changes are already in 
> our build anyway.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13113) upgrade Iceberg from 1.5.0 to 1.5.2

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13113?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13113:

Issue Type: Improvement  (was: Task)

> upgrade Iceberg from 1.5.0 to 1.5.2
> ---
>
> Key: NIFI-13113
> URL: https://issues.apache.org/jira/browse/NIFI-13113
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Sean Sullivan
>Assignee: Joe Witt
>Priority: Minor
> Fix For: 2.0.0-M3
>
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
>  
> Iceberg 1.5.2
>  
> [https://iceberg.apache.org/releases/]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13007) Remove Apache Solr Components

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13007?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13007:

Summary: Remove Apache Solr Components  (was: Remove Solr Components in 2x 
line)

> Remove Apache Solr Components
> -
>
> Key: NIFI-13007
> URL: https://issues.apache.org/jira/browse/NIFI-13007
> Project: Apache NiFi
>  Issue Type: Task
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> In NIFI-12998 it was found that with basic dependency hygiene our Solr 
> components cant even build now.  And it turns out they already dont work at 
> runtime.
> In NIFi-13006 we deprecate and in this JIRA we remove.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12906) Upgrade ZooKeeper to 3.9.2

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12906?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12906:

Issue Type: Improvement  (was: Task)

> Upgrade ZooKeeper to 3.9.2
> --
>
> Key: NIFI-12906
> URL: https://issues.apache.org/jira/browse/NIFI-12906
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 1.26.0, 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-12907) Update to Groovy 4.0.20

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-12907?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-12907:

Issue Type: Improvement  (was: Task)

> Update to Groovy 4.0.20
> ---
>
> Key: NIFI-12907
> URL: https://issues.apache.org/jira/browse/NIFI-12907
> Project: Apache NiFi
>  Issue Type: Improvement
>Reporter: Joe Witt
>Assignee: Joe Witt
>Priority: Major
> Fix For: 2.0.0-M3
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-13211: Decoupling other UIs from exiting NiFi UI [nifi]

2024-05-14 Thread via GitHub


mcgilman commented on PR #8828:
URL: https://github.com/apache/nifi/pull/8828#issuecomment-2110286911

   > the following two files were flagged as missing the standard license header
   
   Thanks @exceptionfactory. I see I have some `rat` config that needs 
updating. New commit coming shortly.


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13228) Jolt Transform UI uses incorrect property names

2024-05-14 Thread David Handermann (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13228?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Handermann updated NIFI-13228:

Affects Version/s: 2.0.0-M2

> Jolt Transform UI uses incorrect property names
> ---
>
> Key: NIFI-13228
> URL: https://issues.apache.org/jira/browse/NIFI-13228
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Extensions
>Affects Versions: 2.0.0-M2
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> NIFI-12554 introducing a breaking change with the property names used by 
> JoltTransformJSON. The existing custom UI saves the transform and 
> specification into properties under the previous name. As a result, the 
> values are stored in dynamic properties. This custom UI needs to be updated 
> to leverage the new/updated Property names.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] NIFI-13211: Decoupling other UIs from exiting NiFi UI [nifi]

2024-05-14 Thread via GitHub


rfellows commented on PR #8828:
URL: https://github.com/apache/nifi/pull/8828#issuecomment-2110255600

   will review...


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Updated] (NIFI-13228) Jolt Transform UI uses incorrect property names

2024-05-14 Thread Matt Gilman (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13228?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Gilman updated NIFI-13228:
---
Status: Patch Available  (was: In Progress)

> Jolt Transform UI uses incorrect property names
> ---
>
> Key: NIFI-13228
> URL: https://issues.apache.org/jira/browse/NIFI-13228
> Project: Apache NiFi
>  Issue Type: Bug
>  Components: Extensions
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> NIFI-12554 introducing a breaking change with the property names used by 
> JoltTransformJSON. The existing custom UI saves the transform and 
> specification into properties under the previous name. As a result, the 
> values are stored in dynamic properties. This custom UI needs to be updated 
> to leverage the new/updated Property names.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13211) Decouple UIs from old NiFi UI

2024-05-14 Thread Matt Gilman (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Gilman updated NIFI-13211:
---
Status: Patch Available  (was: In Progress)

> Decouple UIs from old NiFi UI
> -
>
> Key: NIFI-13211
> URL: https://issues.apache.org/jira/browse/NIFI-13211
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> NiFi is comprised of numerous UIs that all get deployed at start up. These 
> UIs piggybacked on some shared stylesheets, components, and dependencies by 
> directly loading them through a relative path. In preparation for moving away 
> from the existing UI, these UIs are being updated to bundle these items 
> directly. In time, these UIs will all be modernized as well so this is just a 
> transitional step in that process.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13211) Decouple UIs from old NiFi UI

2024-05-14 Thread Matt Gilman (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Gilman updated NIFI-13211:
---
Description: NiFi is comprised of numerous UIs that all get deployed at 
start up. These UIs piggybacked on some shared stylesheets, components, and 
dependencies by directly loading them through a relative path. In preparation 
for moving away from the existing UI, these UIs are being updated to bundle 
these items directly. In time, these UIs will all be modernized as well so this 
is just a transitional step in that process.

> Decouple UIs from old NiFi UI
> -
>
> Key: NIFI-13211
> URL: https://issues.apache.org/jira/browse/NIFI-13211
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> NiFi is comprised of numerous UIs that all get deployed at start up. These 
> UIs piggybacked on some shared stylesheets, components, and dependencies by 
> directly loading them through a relative path. In preparation for moving away 
> from the existing UI, these UIs are being updated to bundle these items 
> directly. In time, these UIs will all be modernized as well so this is just a 
> transitional step in that process.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (NIFI-13211) Decouple UIs from old NiFi UI

2024-05-14 Thread Matt Gilman (Jira)


 [ 
https://issues.apache.org/jira/browse/NIFI-13211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matt Gilman updated NIFI-13211:
---
Summary: Decouple UIs from old NiFi UI  (was: Remove dataviewer 
dependencies on old UI)

> Decouple UIs from old NiFi UI
> -
>
> Key: NIFI-13211
> URL: https://issues.apache.org/jira/browse/NIFI-13211
> Project: Apache NiFi
>  Issue Type: Sub-task
>  Components: Core UI
>Reporter: Matt Gilman
>Assignee: Matt Gilman
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[PR] NIFI-13211: Decoupling other UIs from exiting NiFi UI [nifi]

2024-05-14 Thread via GitHub


mcgilman opened a new pull request, #8828:
URL: https://github.com/apache/nifi/pull/8828

   NIFI-13211:
   
   NiFi is comprised of numerous UIs that all get deployed at start up. These 
UIs piggybacked on some shared stylesheets, components, and dependencies by 
directly loading them through a relative path. In preparation for moving away 
from the existing UI, these UIs are being updated to bundle these items 
directly. In time, these UIs will all be modernized as well so this is just a 
transitional step in that process.


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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (NIFI-13228) Jolt Transform UI uses incorrect property names

2024-05-14 Thread Matt Gilman (Jira)
Matt Gilman created NIFI-13228:
--

 Summary: Jolt Transform UI uses incorrect property names
 Key: NIFI-13228
 URL: https://issues.apache.org/jira/browse/NIFI-13228
 Project: Apache NiFi
  Issue Type: Bug
  Components: Extensions
Reporter: Matt Gilman
Assignee: Matt Gilman


NIFI-12554 introducing a breaking change with the property names used by 
JoltTransformJSON. The existing custom UI saves the transform and specification 
into properties under the previous name. As a result, the values are stored in 
dynamic properties. This custom UI needs to be updated to leverage the 
new/updated Property names.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


Re: [PR] MINIFICPP-2311 Initial support for recordset readers and writers [nifi-minifi-cpp]

2024-05-14 Thread via GitHub


martinzink commented on code in PR #1763:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1763#discussion_r1599894534


##
libminifi/include/controllers/RecordSetWriter.h:
##
@@ -0,0 +1,34 @@
+/**
+* Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+#pragma once
+
+#include "core/controller/ControllerService.h"
+
+#include "core/FlowFile.h"
+#include "core/ProcessSession.h"
+#include "Record.h"
+
+namespace org::apache::nifi::minifi::core {
+
+class RecordSetWriter : public controller::ControllerService {
+ public:
+  using ControllerService::ControllerService;
+
+  virtual void write(const RecordSet& record_set, const 
std::shared_ptr& flow_file, ProcessSession& session) = 0;

Review Comment:
   Unfortunetly we'd need to refactor 
ProcessSession::read/ProcessSession::write for that



##
libminifi/include/controllers/RecordSetReader.h:
##
@@ -0,0 +1,39 @@
+/**
+* Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+#pragma once
+
+
+#include "core/controller/ControllerService.h"
+
+#include "core/FlowFile.h"
+#include "core/ProcessSession.h"
+#include "Record.h"
+#include "utils/Enum.h"
+#include "utils/ProcessorConfigUtils.h"
+
+
+namespace org::apache::nifi::minifi::core {
+
+class RecordSetReader : public controller::ControllerService {
+ public:
+  using ControllerService::ControllerService;
+
+
+  virtual nonstd::expected read(const 
std::shared_ptr& flow_file, ProcessSession& session) = 0;

Review Comment:
   Unfortunetly we'd need to refactor 
ProcessSession::read/ProcessSession::write for that



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

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (NIFI-13227) Bump aspectj and mockito and various dependencies

2024-05-14 Thread Joe Witt (Jira)
Joe Witt created NIFI-13227:
---

 Summary: Bump aspectj and mockito and various dependencies
 Key: NIFI-13227
 URL: https://issues.apache.org/jira/browse/NIFI-13227
 Project: Apache NiFi
  Issue Type: Task
Reporter: Joe Witt
Assignee: Joe Witt


aspectj 1.9.22.1
mockito 5.12.0
software.amazon.awssdk 2.25.51
com.amazonaws 1.12.721
com.slack.api 1.39.2




--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (NIFI-12812) CaptureChangeMySQL consumes binlog backlog

2024-05-14 Thread Joe Witt (Jira)


[ 
https://issues.apache.org/jira/browse/NIFI-12812?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17846189#comment-17846189
 ] 

Joe Witt commented on NIFI-12812:
-

User xiyang from apache slack states:
I used CaptureChangeMySQL 1.23.3 for the consumption of binlogs. In the long 
running process, we found that there will often be stagnation (we did the 
monitoring, extracted the binlog position of the main database and the 
comparison of binlog.position), Delayed consumption or backlog occurs; Or in 
the case of a large number of updates and inserts, binlog.filename may be 
backlogged by 10 or more. Is there any way to optimize this, thanks!



We analyze that setting Events Per FlowFile too small will cause binlog 
consumption backlog, setting Events Per FlowFile too small will cause data to 
be delayed indefinitely, which is not acceptable in cdc. When the Events Per 
FlowFile is set to 150, the data backlog can reach 2 minutes, which is a 
serious problem that we hope can be fixed


After nearly two months of observation, we found that there will be a 
phenomenon when the consumption binlog is stuck, the running thread identifier 
on the CaptureChangeMySQL processor is always 1, theoretically the running 
thread identifier should disappear soon, when we find the binlog backlog, the 
running thread identifier will always be there, at this time, When we look at 
CaptureChangeMySQL consumption capabilities, it's very slow. I'm not sure 
what's wrong with it. When we stop the processor, or replace it with another 
processor, it runs for a long time, which confuses me.


CPU:
Architecture:  x86_64
CPU op-mode(s):32-bit, 64-bit
Byte Order:Little Endian
CPU(s):16
On-line CPU(s) list:   0-15
Thread(s) per core:2
Core(s) per socket:8
Socket(s): 1
NUMA node(s):  1
Vendor ID: AuthenticAMD
CPU family:25
Model: 1
Model name:AMD EPYC 7T83 64-Core Processor
Stepping:  1
CPU MHz:   2545.218
BogoMIPS:  5090.43
Hypervisor vendor: KVM
Virtualization type:   full
L1d cache: 32K
L1i cache: 32K
L2 cache:  512K
L3 cache:  32768K
NUMA node0 CPU(s): 0-15
memory
32G
java.arg.2=-Xms1g
java.arg.3=-Xmx16g
disk
200G SSD * 2

--NiFi BinLog monitor alarm --
NiFi data synchronization is abnormal!!
Corresponding error message type: 2
Corresponding error message: binlog Position backlog absolute value exceeds 
configuration value :(1000), backlog more than 10 times, please check!
Processor Id: f14230db-ba78-106d-973f-e90bd03f0a28
Processor Name: Advertising link -cdc
mysql: mysql-bin.262489
Corresponding point in the processor: mysql-bin.262489
Backlog value: 99644822
Current error accumulation: 11
Notice time: 2024-04-24 00:18:00
This is yesterday's alarm information, let me explain: the binlog file of the 
current processor and the database is the same, the backlog value is 99644822, 
it will be compared every 1 minute, if more than 1000, +1, when more than 
10 times, send alarm information; We can see that the current backlog is 11 
times, which is 11 minutes, and the consumption rate of nifi has not caught up 
with the production rate of mysql. But when we stop the processor and change 
the value of Events Per FlowFile symbolically (from 5 to 10, or whatever), it 
runs normally for a while. Until the next backlog occurs

 I initially suspected a thread related problem, but I'm not sure which step, 
when a processor will be called to run in NIFI, when the UI shows a thread 
identifier on the processor, this processor should be running at this time, but 
when the thread identifier in UI is always 1, where will the processor be 
stuck? If we solve this problem, we will find the truth. (Forgive me if I don't 
know much about how NIFI works, or if it has anything to do with Penalty 
Duration and Yield Duration.)






@Joe Witt
 The problem is repeated, and the relevant log is obtained in time. Let me 
describe the file. The file with restart-after is the log information of the 
restart after the problem (restarting the processor, the task is normal), and 
the file without restart-after is the log information obtained when the problem 
occurs. Problems of the database instance is 
pxc-hzr66p3487fwri.polarx.rds.aliyuncs.com, seemingly a lock, but I am not sure 
what causes it.(The number after each file, representing minutes, is executed 
every minute)

xiyang
  19 hours ago
public void onEvent(Event event) {
RawBinlogEvent ep = new RawBinlogEvent(event, client.getBinlogFilename());
try {
while (!stopNow.get()) {
if (queue.offer(ep, QUEUE_OFFER_TIMEOUT_MSEC, 
TimeUnit.MILLISECONDS)) {
return;
}
}

logger.info("Stopped while waiting to enqueue event");
} catch