Refactor Serialize logic to produce correct strings.

Removes the dependency on the `url` crate because we only need
the percent decoding, with our own encoding set.

All keys/values are conservatively percent-encoded, but the square
brackets are untouched. Compatible with strict decoding, and
potentially less error prone.

1 files changed, 17 insertions, 0 deletions

diff --git a/tests/test_deserialize.rs b/tests/test_deserialize.rs
index e0392f5..a71e4bf 100644
--- a/tests/test_deserialize.rs
+++ b/tests/test_deserialize.rs
@@ -443,4 +443,21 @@ fn strict_mode() {
 
     let params: Result<Query, _> = loose_config.deserialize_str("vec%5B0%5D%5Ba%5D]=1&vec[1][a]=2");
     assert_eq!(params.unwrap(), Query { vec: vec![Test { a: 1 }, Test { a: 2 }] });
+
+    #[derive(Deserialize,Serialize,Debug, PartialEq)]
+    struct OddTest {
+        #[serde(rename="[but&why=?]")]
+        a: u8
+    }
+
+    let params = OddTest { a: 12 };
+    let enc_params = qs::to_string(&params).unwrap();
+    println!("Enocded as: {}", enc_params);
+    let rec_params: Result<OddTest, _> = strict_config.deserialize_str(&enc_params);
+    assert_eq!(rec_params.unwrap(), params);
+
+    // Non-strict decoding cannot necessarily handle these weird scenerios.
+    let rec_params: Result<OddTest, _> = loose_config.deserialize_str(&enc_params);
+    assert!(rec_params.is_err());
+    println!("{}", rec_params.unwrap_err());
 }
\ No newline at end of file