[ 
https://issues.apache.org/jira/browse/SCM-992?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17571213#comment-17571213
 ] 

ASF GitHub Bot commented on SCM-992:
------------------------------------

kwin commented on code in PR #159:
URL: https://github.com/apache/maven-scm/pull/159#discussion_r929566524


##########
maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/src/main/java/org/apache/maven/scm/provider/git/gitexe/command/GitCommandLineUtils.java:
##########
@@ -196,4 +209,27 @@ public static int execute( Commandline cl, 
CommandLineUtils.StringStreamConsumer
         return exitCode;
     }
 
+    static Map<String, String> prepareEnvVariablesForRepository( 
GitScmProviderRepository repository,
+                                                                 Map<String, 
String> environmentVariables )
+    {
+        Map<String, String> effectiveEnvironmentVariables = new HashMap<>();
+        if ( environmentVariables != null )
+        {
+            effectiveEnvironmentVariables.putAll( environmentVariables );
+        }
+        if ( StringUtils.isNotBlank( repository.getPrivateKey() ) )
+        {
+            if ( effectiveEnvironmentVariables.putIfAbsent( 
VARIABLE_GIT_SSH_COMMAND, "ssh -IdentitiesOnly=yes -i "

Review Comment:
   Good catch, fixed in 
https://github.com/apache/maven-scm/pull/159/commits/42722d952a03b2dc80e981d12c8c5aa8dba3cf87.
 





> Support explicitly configured SSH private key for gitexe
> --------------------------------------------------------
>
>                 Key: SCM-992
>                 URL: https://issues.apache.org/jira/browse/SCM-992
>             Project: Maven SCM
>          Issue Type: Improvement
>          Components: maven-scm-provider-gitexe
>            Reporter: Konrad Windszus
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 2.0.0-M2
>
>
> Currently the {{privateKey}} parameter set on 
> {{ScmProviderRepositoryWithHost}} is only evaluated in provider "jgit". It 
> should also be used for provider "git". Same for {{passphrase}} parameter.



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

Reply via email to