[
https://issues.apache.org/jira/browse/SCM-415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17961342#comment-17961342
]
ASF GitHub Bot commented on SCM-415:
------------------------------------
jira-importer commented on issue #626:
URL: https://github.com/apache/maven-scm/issues/626#issuecomment-2964607591
**[ajbanck](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=ajbanck)**
commented
Second patch also works for me.
> 1.1 requires setting a Perforce password breaking Perforce setup for external
> authentication
> --------------------------------------------------------------------------------------------
>
> Key: SCM-415
> URL: https://issues.apache.org/jira/browse/SCM-415
> Project: Maven SCM (Moved to GitHub Issues)
> Issue Type: Bug
> Components: maven-scm-provider-perforce
> Affects Versions: 1.1
> Reporter: ajbanck
> Assignee: Carlos Sanchez Gonzalez
> Priority: Blocker
> Fix For: 1.2
>
> Attachments: perforce_login.patch, scm-415.patch
>
>
> checkin r654744 for SCM-363 introduces a requirement to set a Perforce
> password.
> This will break the SCM provider when Perforce is configured to not accept a
> password. This will be the case when using the OS security instead of
> Perforce' own.
> The SCM pluging throws an exception when not providing a password, Perforce
> throws an error when trying to login with a (dummy) password.
> When no username and/or password is provided, the Perforce login step should
> be skipped.
> =========
> Perforce login output when running p4 from the command prompt:
> p4 login
> 'login' not necessary, no password set for this user.
> p4 login john
> You don't have permission for this operation.
> =========
> Offending code:
> PerforceLoginCommand.java
> if ( StringUtils.isEmpty( repo.getPassword() ) )
> {
> throw new ScmException( "password is required for the
> perforce scm plugin." );
> }
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
