You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Cristian Lupu (JIRA)" <ji...@codehaus.org> on 2012/02/06 17:52:03 UTC

[jira] (MCHANGES-218) Downloading from JIRA fails if issueManagement.url has trailing slash

    [ https://jira.codehaus.org/browse/MCHANGES-218?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=290908#comment-290908 ] 

Cristian Lupu commented on MCHANGES-218:
----------------------------------------

This fix doesn't work for the case when url is set to a value like "http://jira.codehaus.org/"
Because it will remove the last "/" and few lines below it tries to extract url again based on the last index for "/" and the url extracted in that case is "http:/"

54          if ( url.endsWith( "/" ) )
55          {
56              // MCHANGES-218
57              url = url.substring( 0, url.lastIndexOf( '/' ) );
58          }
.....................
72          String jiraUrl = url.substring( 0, url.lastIndexOf( '/' ) );

Easy to test with a quick unit test and a different value for issueManagementUrl:
Map<String, String> jiraUrlAndProjectId = getJiraUrlAndProjectId("http://jira.codehaus.org/");
assertEquals("http://jira.codehaus.org", jiraUrlAndProjectId.get("url"));





                
> Downloading from JIRA fails if issueManagement.url has trailing slash
> ---------------------------------------------------------------------
>
>                 Key: MCHANGES-218
>                 URL: https://jira.codehaus.org/browse/MCHANGES-218
>             Project: Maven 2.x Changes Plugin
>          Issue Type: Bug
>          Components: jira
>    Affects Versions: 2.3
>            Reporter: Lukas Theussl
>            Assignee: Lukas Theussl
>             Fix For: 2.4
>
>
> With an issueManagement.url like http://jira.codehaus.org/browse/DOXIA/ (note trailing slash), the jira report chokes
> {noformat}
> [WARNING] Downloading from JIRA failed. Received: [404]
> {noformat}
> and an empty report is generated. removing the trailing slash makes it work.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira