paddle: Require that input `Iterator`s also be `PartialOrd`

In order to properly verify the signature, dictionary entries must be
serialized in sorted order. Seems simpler to put the onus on the caller
to ensure the entries can be sorted rather than having to deal with that
myself.

1 files changed, 2 insertions, 2 deletions

diff --git a/license-generator/paddle/src/lib.rs b/license-generator/paddle/src/lib.rs
index 5aef6fe..3ebc3f2 100644
--- a/license-generator/paddle/src/lib.rs
+++ b/license-generator/paddle/src/lib.rs
@@ -8,7 +8,7 @@ use openssl::sign::Verifier;
 
 // https://paddle.com/docs/reference-verifying-webhooks/
 fn verify_signature<'a, I>(pem: &[u8], signature: &str, params: I) -> bool
-where I: IntoIterator<Item = (&'a str, &'a str)> {
+where I: IntoIterator<Item = (&'a str, &'a str)> + PartialOrd {
     let rsa = Rsa::public_key_from_pem(pem).unwrap();
     let pkey = PKey::from_rsa(rsa).unwrap();
     let mut verifier = Verifier::new(MessageDigest::sha1(), &pkey).unwrap();
@@ -20,7 +20,7 @@ where I: IntoIterator<Item = (&'a str, &'a str)> {
 }
 
 fn php_serialize<'a, I>(pairs: I) -> String
-where I: IntoIterator<Item = (&'a str, &'a str)> {
+where I: IntoIterator<Item = (&'a str, &'a str)> + PartialOrd {
     let mut serialized = String::with_capacity(500);
 
     let mut len = 0;