In opendal's code, there are many places need to build query pairs:

However, many of them are poorly implemented, require unnecessary error handling, or consume additional memory.

This refactor intended to use newly added QueryPairsWriter to replace them all:

- let mut url = format!("{}?list-type=2", self.endpoint);
- if !p.is_empty() {
-     write!(url, "&prefix={}", percent_encode_path(&p))
-         .expect("write into string must succeed");
- }
+ let mut url = QueryPairsWriter::new(&self.endpoint);
+ url = url.push("list-type", "2");
+ if !p.is_empty() {
+     url = url.push("prefix", &percent_encode_path(&p));
+ }

Take #5977 as a full example. We don't need to replace every query build—only those with more than three query pairs, such as the list object operations. Please only migrate list operation.

Tasks

• s3
• aliyun_drive
• azblob
• azdls
• azfile
• b2
• cos
• gcs
• gdrive
• obs
• onedrive
• oss
• swift