volkodava opened a new pull request #74:
URL: https://github.com/apache/maven-wagon/pull/74


   In this pull request we have a fix for breaking change introduced by 
`EncodingUtil.encodeURLToString( String baseUrl, String path )` for URLs with 
trailing slash in the following commit: 
https://github.com/apache/maven-wagon/commit/7d1315e47df948d8b17f5c759b20a5b3437cbff7#diff-a0742c473bc4d821caf820463c87956b67da4d07c059d5b6a631e98dd3f7f7b4R93,
 which has two issues:
   - inconsistency between `EncodingUtil.encodeURLToString( String url )` and 
`EncodingUtil.encodeURLToString( String baseUrl, String path )` in regards to 
URLs with trailing slash
   - affects 
`org.apache.maven.wagon.shared.http.AbstractHttpClientWagon.resourceExists(...)`
 API, as URL `https://host:1234/test/demo.zip/` gets normalized to 
`https://host:1234/test/demo.zip` (with no trailing slash) by 
`EncodingUtil.encodeURLToString( String baseUrl, String path )` before verify 
if resource exists


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to