michael-o edited a comment on issue #277: [MNG-6732] - 
DefaultArtifactDescriptorReader.loadPom to check IGNORE_MISSING policy upon 
ArtifactTransferException
URL: https://github.com/apache/maven/pull/277#issuecomment-575775520
 
 
   Here is the next problem: IT for MNG-3470 fails, see 
http://home.apache.org/~michaelo/issues/MNG-6732/IT-3470/. The checksum 
validation is ignored because the `ChecksumFailureException` is wrapped inside 
an `ArtifactTransferException`. The problem is even if we could determine 
wether it is `IGNORE_MISSING` or `IGNORE_IVALID` in this case and the checksum 
policy would be `fail`, but the `ArtifactDescriptorPolicy` is lax, we have a 
conflict. The checksum policy needs to beat the artifact descriptor policy. 
This is a hard nut to crack. Comments?
   I believe that the current code is broken and your PR simply reveals this 
bug.
   
   This is our problem in 
`org.apache.maven.repository.internal.MavenRepositorySystemUtils`:
   ```
   session.setArtifactDescriptorPolicy( new SimpleArtifactDescriptorPolicy( 
true, true ) );
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to