[jira] Commented: (MRM-154) DigestUtils fails to verify checksum from ibiblio

["Aaron Digulla (JIRA)" <ji...@codehaus.org>]

    [ http://jira.codehaus.org/browse/MRM-154?page=comments#action_81156 ] 
            
Aaron Digulla commented on MRM-154:
-----------------------------------

The supplied patch fails if the filename is "-" (stdin) like in http://repo1.maven.org/maven2/org/apache/maven/archetype/maven-archetype/1.0-alpha-4/maven-archetype-1.0-alpha-4.pom.md5:

54210dc92a25325496713d19d5ced2b9  -


> DigestUtils fails to verify checksum from ibiblio
> -------------------------------------------------
>
>                 Key: MRM-154
>                 URL: http://jira.codehaus.org/browse/MRM-154
>             Project: Archiva
>          Issue Type: Bug
>            Reporter: nicolas de loof
>         Assigned To: Brett Porter
>             Fix For: 1.0-beta-1
>
>         Attachments: DigestUtils.java.patch, MRM-154.patch
>
>
> Downloading servletapi-24.pom fails.
> DigestUtils.cleanChecksum check for filename in remote checksum file to be same as local, but tets is inverted :
>                 String filename = m.group( 2 );
>                 if ( !path.endsWith( filename ) )
>                 {
>                     throw new DigesterException( "Supplied checksum does not match checksum pattern" );
>                 }
> filename = "/home/projects/maven/repository-staging/to-ibiblio/maven2/servletapi/servletapi/2.4/servletapi-2.4.pom"
> path = "servletapi-2.4.pom".
> Patch provided to test if ( !path.endsWith( filename ) )

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira