diff --git a/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java b/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java index 68ab280..db1b922 100644 --- a/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java +++ b/src/main/java/org/apache/maven/plugins/gpg/AbstractGpgMojo.java @@ -337,20 +337,7 @@ private void logBestPracticeWarning(String source) { } protected AbstractGpgSigner newSigner(MavenProject mavenProject) throws MojoFailureException { - AbstractGpgSigner signer; - if (GpgSigner.NAME.equals(this.signer)) { - signer = new GpgSigner(executable); - } else if (BcSigner.NAME.equals(this.signer)) { - signer = new BcSigner( - session.getRepositorySession(), - keyEnvName, - keyFingerprintEnvName, - agentSocketLocations, - keyFilePath, - keyFingerprint); - } else { - throw new MojoFailureException("Unknown signer: " + this.signer); - } + AbstractGpgSigner signer = createSigner(this.signer); signer.setLog(getLog()); signer.setInteractive(settings.isInteractiveMode()); @@ -395,6 +382,24 @@ protected AbstractGpgSigner newSigner(MavenProject mavenProject) throws MojoFail return signer; } + protected AbstractGpgSigner createSigner(String name) throws MojoFailureException { + AbstractGpgSigner signer; + if (GpgSigner.NAME.equals(name)) { + signer = new GpgSigner(executable); + } else if (BcSigner.NAME.equals(name)) { + signer = new BcSigner( + session.getRepositorySession(), + keyEnvName, + keyFingerprintEnvName, + agentSocketLocations, + keyFilePath, + keyFingerprint); + } else { + throw new MojoFailureException("Unknown signer: " + name); + } + return signer; + } + private boolean isNotBlank(String string) { return string != null && !string.trim().isEmpty(); }