AdamGS commented on code in PR #334:
URL:
https://github.com/apache/arrow-rs-object-store/pull/334#discussion_r2045308481
##########
src/client/builder.rs:
##########
@@ -183,7 +183,8 @@ impl HttpRequestBuilder {
let mut error = None;
if let Ok(ref mut req) = self.request {
let mut out = format!("{}?", req.uri().path());
- let mut encoder = form_urlencoded::Serializer::new(&mut out);
+ let start_position = out.len();
Review Comment:
yeah, after this change the URL will again be
`azurebase.com/path?comp=list&restype:container`.
Part of the problem here is that I realized that it seems like [Azurite
behaves differently](https://github.com/Azure/Azurite/issues/2548) than Azure
in this case, so tests that will fail on the real Azure (like @a10y's original
repro) would pass in the repo's test suite.
--
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]
