This is an automated email from the ASF dual-hosted git repository.
exceptionfactory pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/nifi.git
The following commit(s) were added to refs/heads/main by this push:
new 49702f1f51 NIFI-12668 Fix conflict in Registry Git provider with
gpg.format=ssh
49702f1f51 is described below
commit 49702f1f510cc33c9cb52bb5b9580009a1d8afe8
Author: Brian Ghigiarelli <[email protected]>
AuthorDate: Wed Jan 24 22:21:27 2024 -0500
NIFI-12668 Fix conflict in Registry Git provider with gpg.format=ssh
NIFI-12667 Upgraded JGit from 6.7.0 to 6.8.0
This closes #8300
Signed-off-by: David Handermann <[email protected]>
---
.../apache/nifi/registry/provider/flow/git/GitFlowMetaData.java | 9 +++++++++
nifi-registry/pom.xml | 2 +-
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git
a/nifi-registry/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/flow/git/GitFlowMetaData.java
b/nifi-registry/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/flow/git/GitFlowMetaData.java
index 5e99c392eb..5c7720921d 100644
---
a/nifi-registry/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/flow/git/GitFlowMetaData.java
+++
b/nifi-registry/nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/provider/flow/git/GitFlowMetaData.java
@@ -24,6 +24,8 @@ import org.eclipse.jgit.api.PushCommand;
import org.eclipse.jgit.api.Status;
import org.eclipse.jgit.api.errors.GitAPIException;
import org.eclipse.jgit.api.errors.NoHeadException;
+import org.eclipse.jgit.lib.Config;
+import org.eclipse.jgit.lib.GpgConfig;
import org.eclipse.jgit.lib.ObjectId;
import org.eclipse.jgit.lib.ObjectStream;
import org.eclipse.jgit.lib.Ref;
@@ -475,9 +477,16 @@ class GitFlowMetaData {
final String commitMessage = isEmpty(author) ? message
: format("%s\n\nBy NiFi Registry user: %s", message,
author);
+
+ // Ensure that we are providing a valid GPG Format to jgit, even
though
+ // it is not used for signing. This avoids an error if the system's
+ // git config for gpg.format is "ssh".
+ final Config unusedConfig = new Config();
+ unusedConfig.setEnum("gpg", null, "format",
GpgConfig.GpgFormat.OPENPGP);
final RevCommit commit = git.commit()
.setMessage(commitMessage)
.setSign(false)
+ .setGpgConfig(new GpgConfig(unusedConfig))
.call();
if (flowPointer != null) {
diff --git a/nifi-registry/pom.xml b/nifi-registry/pom.xml
index b4356cbd43..2fd110ea25 100644
--- a/nifi-registry/pom.xml
+++ b/nifi-registry/pom.xml
@@ -40,7 +40,7 @@
<flyway.version>9.22.3</flyway.version>
<flyway.tests.version>9.5.0</flyway.tests.version>
<swagger.ui.version>3.12.0</swagger.ui.version>
- <jgit.version>6.7.0.202309050840-r</jgit.version>
+ <jgit.version>6.8.0.202311291450-r</jgit.version>
<org.apache.sshd.version>2.12.0</org.apache.sshd.version>
</properties>