Request: use http-types for querystring handling

It seems like this was overlooked from
b6113de

Author: Fishrock123

Cargo.toml

@@ -34,7 +34,6 @@ log = { version = "0.4.7", features = ["kv_unstable"] }
 mime_guess = "2.0.3"
 serde = "1.0.97"
 serde_json = "1.0.40"
-serde_urlencoded = "0.6.1"
 url = "2.0.0"
 http-client = { version = "6.0.0", default-features = false }
 http-types = "2.0.0"

src/request.rs

@@ -1,7 +1,7 @@
 use crate::http::{
     self,
     headers::{self, HeaderName, HeaderValues, ToHeaderValues},
-    Body, Error, Method, Mime,
+    Body, Method, Mime,
 };
 use crate::RequestBuilder;
 
@@ -89,14 +89,8 @@ impl Request {
     /// assert_eq!(page, 2);
     /// # Ok(()) }
     /// ```
-    pub fn query<T: serde::de::DeserializeOwned>(&self) -> Result<T, Error> {
-        use std::io::{Error, ErrorKind};
-        let query = self
-            .req
-            .url()
-            .query()
-            .ok_or_else(|| Error::from(ErrorKind::InvalidData))?;
-        Ok(serde_urlencoded::from_str(query)?)
+    pub fn query<T: serde::de::DeserializeOwned>(&self) -> crate::Result<T> {
+        self.req.query()
     }
 
     /// Set the URL querystring.
@@ -119,16 +113,8 @@ impl Request {
     /// assert_eq!(req.as_ref().url().as_str(), "https://httpbin.org/get?page=2");
     /// # Ok(()) }
     /// ```
-    pub fn set_query(
-        &mut self,
-        query: &(impl Serialize + ?Sized),
-    ) -> Result<(), serde_urlencoded::ser::Error> {
-        let query = serde_urlencoded::to_string(query)?;
-        self.req.url_mut().set_query(Some(&query));
-
-        *self.req.url_mut() = self.req.url().clone();
-
-        Ok(())
+    pub fn set_query(&mut self, query: &impl Serialize) -> crate::Result<()> {
+        self.req.set_query(query)
     }
 
     /// Get an HTTP header.

src/request_builder.rs

@@ -2,7 +2,7 @@ use crate::http::{
     headers::{HeaderName, ToHeaderValues},
     Body, Method, Mime,
 };
-use crate::{Client, Request, Response, Result};
+use crate::{Client, Error, Request, Response, Result};
 
 use futures_util::future::BoxFuture;
 use serde::Serialize;
@@ -161,10 +161,7 @@ impl RequestBuilder {
     /// assert_eq!(req.as_ref().url().as_str(), "https://httpbin.org/get?page=2");
     /// # Ok(()) }
     /// ```
-    pub fn query(
-        mut self,
-        query: &(impl Serialize + ?Sized),
-    ) -> std::result::Result<Self, serde_urlencoded::ser::Error> {
+    pub fn query(mut self, query: &impl Serialize) -> std::result::Result<Self, Error> {
         self.req.as_mut().unwrap().set_query(query)?;
 
         Ok(self)