tustvold commented on code in PR #435: URL: https://github.com/apache/arrow-rs-object-store/pull/435#discussion_r2199018452
########## src/client/builder.rs: ########## @@ -181,9 +181,28 @@ impl HttpRequestBuilder { pub(crate) fn query<T: serde::Serialize + ?Sized>(mut self, query: &T) -> Self { let mut error = None; if let Ok(ref mut req) = self.request { - let mut out = format!("{}?", req.uri().path()); - let start_position = out.len(); - let mut encoder = form_urlencoded::Serializer::for_suffix(&mut out, start_position); + // Build the query string, preserving any existing query parameters. + // This prevents malformed URIs like "/bucket?&list-type=2" when the URI + // already contains query parameters. Review Comment: Why do we need to handle this case, we didn't before - we solely used the path? Was this AI generated or something? -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org