slachiewicz opened a new pull request, #2126:
URL: https://github.com/apache/maven-resolver/pull/2126

   Ports the connector part of #2084 to the 1.9.x line. `master` keeps its keys 
in `BasicRepositoryConnectorConfigurationKeys`; 1.9.x uses literal keys in the 
connector, so the property is added there as 
`aether.connector.basic.failOnChecksumUploadFailure`, with the usual repository 
id suffix.
   
   `uploadChecksum` caught every exception, logged at WARN, and let the PUT 
complete, so a deploy whose checksum uploads were rejected reported success 
with the artifact published and nothing next to it to verify against. The 
failure now propagates and fails the transfer. Repositories that refuse 
checksum PUTs can opt out per repository or globally by setting the property to 
`false`, which restores the WARN-only behavior.
   
   No new test: the connector has no transporter-level test harness on this 
line, and `master`'s change added none either. The doc row is added to 
`configuration.md`.
   
   Verified: `mvn -pl maven-resolver-connector-basic -am verify` -> BUILD 
SUCCESS, 23 tests in the module, 0 failures; `spotless:apply` produced no 
further changes.
   
   *This change was created with AI assistance.*
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to