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

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

jira-importer opened a new issue, #1140:
URL: https://github.com/apache/maven-scm/issues/1140

   **[Michael 
Osipov](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=michael-o)**
 opened 
**[SCM-909](https://issues.apache.org/jira/browse/SCM-909?redirect=false)** and 
commented
   
   This issue was been found on one of Jenkins slaves. Consider the following 
path: 
`F:\jenkins\jenkins-slave\workspace\maven-box_maven-scm_SCM-906-3AXCXWOA5KAAL37TCEKGJUQCHVU3HMOGCVV63JP76C24C3X6DHOA@2\m\maven-scm-providers\maven-scm-providers-git\maven-scm-provider-gitexe\target`.
 THe aforementioned class outputs:
   
   ```
   
file:///F:/jenkins/jenkins-slave/workspace/maven-box_maven-scm_SCM-906-3AXCXWOA5KAAL37TCEKGJUQCHVU3HMOGCVV63JP76C24C3X6DHOA@2/m/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/target
   
file://2/m/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/target
   ```
   
   The parse logic incorrectly assumes that the last `@` is for the user info. 
According to RFC 8089 a file URL may contain this information, but our parser 
would still fail becuase it uses `indexOf`. It must stop searching to the first 
slash.
   
   Note: the example is from the GitExe provider, but it also applies to SvnExe.
   
   
   ---
   
   **Affects:** 1.10.0
   
   **Sub-tasks:**
   - [SCM-910](https://issues.apache.org/jira/browse/SCM-910) 
SvnScmProviderRepository#parseUrl(): skip parsing of user info for file:// URLs
   
   




> SvnScmProviderRepository#parseUrl() does not comply with RFC 3986 and RFC 8089
> ------------------------------------------------------------------------------
>
>                 Key: SCM-909
>                 URL: https://issues.apache.org/jira/browse/SCM-909
>             Project: Maven SCM (Moved to GitHub Issues)
>          Issue Type: Bug
>          Components: maven-scm-provider-svn
>    Affects Versions: 1.10.0
>            Reporter: Michael Osipov
>            Priority: Major
>
> This issue was been found on one of Jenkins slaves. Consider the following 
> path: 
> {{F:\jenkins\jenkins-slave\workspace\maven-box_maven-scm_SCM-906-3AXCXWOA5KAAL37TCEKGJUQCHVU3HMOGCVV63JP76C24C3X6DHOA@2\m\maven-scm-providers\maven-scm-providers-git\maven-scm-provider-gitexe\target}}.
>  THe aforementioned class outputs:
> {noformat}
> file:///F:/jenkins/jenkins-slave/workspace/maven-box_maven-scm_SCM-906-3AXCXWOA5KAAL37TCEKGJUQCHVU3HMOGCVV63JP76C24C3X6DHOA@2/m/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/target
> file://2/m/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gitexe/target
> {noformat}
> The parse logic incorrectly assumes that the last {{@}} is for the user info. 
> According to RFC 8089 a file URL may contain this information, but our parser 
> would still fail becuase it uses {{indexOf}}. It must stop searching to the 
> first slash.
> Note: the example is from the GitExe provider, but it also applies to SvnExe.



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

Reply via email to