ChJung1 commented on PR #155: URL: https://github.com/apache/jclouds/pull/155#issuecomment-1650484853
Hi Steffen Gürtler, I did the pull request for the encoding with Base64Url, because with Base64 i got an error similar to yours. Base64Url is meant to ensure you can use the string in urls. The string you have contains a minus sign, so this is indeed Base64Url, and not Base64, so my fix is in there. Your problem looks very similar to mine, so is it maybe the "==" this time? BTW, while changing the code i wondered if we could implement makeBlockId() by using String.valueOf() instead, which is much simpler code, a shorter string, and uses only digits (as Block-ID is an integer and probably positive) Feel free to try that, and if it helps you can maybe also remove that not-so-helpful test I added. Andrew Gaul will then hopefully approve your pull request. Good luck! ChJung1 Gesendet: Dienstag, 25. Juli 2023 um 18:07 Uhr Von: "Steffen Gürtler" ***@***.***> An: "apache/jclouds" ***@***.***> Cc: "ChJung1" ***@***.***>, "Mention" ***@***.***> Betreff: Re: [apache/jclouds] Jclouds 1615 (PR #155) I have tested some simple things with the current JCloud master branch code base against an Azure storage account and run into an error message for a 4 gigabyte upload, split into 5 MByte chunks in a multipart upload: Could it be, that the base64 encoding in the url version isn't accepted by Azure? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: ***@***.***> -- 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: notifications-unsubscr...@jclouds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
