diff --git a/examples/cosign/verify/main.rs b/examples/cosign/verify/main.rs index 4295c60222..b99e59d955 100644 --- a/examples/cosign/verify/main.rs +++ b/examples/cosign/verify/main.rs @@ -111,7 +111,7 @@ async fn run_app( let mut client_builder = sigstore::cosign::ClientBuilder::default(); if let Some(key) = frd.rekor_pub_key.as_ref() { - client_builder = client_builder.with_rekor_pub_key(&key); + client_builder = client_builder.with_rekor_pub_key(key); } if !frd.fulcio_certs.is_empty() { diff --git a/examples/openidflow/openidconnect/main.rs b/examples/openidflow/openidconnect/main.rs index 87d14b0f98..4db193715d 100644 --- a/examples/openidflow/openidconnect/main.rs +++ b/examples/openidflow/openidconnect/main.rs @@ -30,7 +30,7 @@ fn main() -> Result<(), anyhow::Error> { open::that(url.0.to_string())?; println!( "Open this URL in a browser if it does not automatically open for you:\n{}\n", - url.0.to_string() + url.0 ); } Err(e) => println!("{}", e), diff --git a/src/crypto/mod.rs b/src/crypto/mod.rs index 5be3d21289..10cbe61fc6 100644 --- a/src/crypto/mod.rs +++ b/src/crypto/mod.rs @@ -238,7 +238,7 @@ OSWS1X9vPavpiQOoTTGC0xX57OojUadxF1cdQmrsiReWg2Wn4FneJfa8xw== let public_key = private_key.public_key(); let ec_pub_key = - EcKey::from_public_key(&group, &public_key).expect("Cannot create ec pub key"); + EcKey::from_public_key(&group, public_key).expect("Cannot create ec pub key"); let pkey = pkey::PKey::from_ec_key(ec_pub_key).expect("Cannot create pkey"); let mut x509_name_builder = X509NameBuilder::new()?; @@ -369,7 +369,7 @@ OSWS1X9vPavpiQOoTTGC0xX57OojUadxF1cdQmrsiReWg2Wn4FneJfa8xw== // set issuer if let Some(issuer_data) = issuer { let issuer_name = issuer_data.cert.subject_name(); - x509_builder.set_issuer_name(&issuer_name)?; + x509_builder.set_issuer_name(issuer_name)?; } else { // self signed cert x509_builder.set_issuer_name(&x509_name)?; diff --git a/src/crypto/signing_key/mod.rs b/src/crypto/signing_key/mod.rs index cc324297d3..d2cce7ceda 100644 --- a/src/crypto/signing_key/mod.rs +++ b/src/crypto/signing_key/mod.rs @@ -340,10 +340,8 @@ mod tests { #[case(SigningScheme::ECDSA_P384_SHA384_ASN1)] #[case(SigningScheme::ED25519)] fn sigstore_signing(#[case] signing_scheme: SigningScheme) { - let signer = signing_scheme.create_signer().expect(&format!( - "create SigStoreSigner with {:?} failed", - signing_scheme - )); + let signer = signing_scheme.create_signer().unwrap_or_else(|_| panic!("create SigStoreSigner with {:?} failed", + signing_scheme)); let key_pair = signer .to_sigstore_keypair() .expect("convert SigStoreSigner to SigStoreKeypair failed."); diff --git a/src/crypto/verification_key.rs b/src/crypto/verification_key.rs index 9a6cbb5f55..be81107107 100644 --- a/src/crypto/verification_key.rs +++ b/src/crypto/verification_key.rs @@ -288,7 +288,7 @@ mod tests { .expect("Cannot create CosignVerificationKey"); let msg = r#"{"critical":{"identity":{"docker-reference":"registry-testing.svc.lan/busybox"},"image":{"docker-manifest-digest":"sha256:f3cfc9d0dbf931d3db4685ec659b7ac68e2a578219da4aae65427886e649b06b"},"type":"cosign container image signature"},"optional":null}"#; - let outcome = verification_key.verify_signature(signature, &msg.as_bytes()); + let outcome = verification_key.verify_signature(signature, msg.as_bytes()); assert!(outcome.is_ok()); } @@ -301,7 +301,7 @@ mod tests { let msg = "hello world"; let err = verification_key - .verify_signature(signature, &msg.as_bytes()) + .verify_signature(signature, msg.as_bytes()) .expect_err("Was expecting an error"); let found = match err { SigstoreError::PublicKeyVerificationError => true, @@ -319,7 +319,7 @@ mod tests { let msg = r#"{"critical":{"identity":{"docker-reference":"registry-testing.svc.lan/busybox"},"image":{"docker-manifest-digest":"sha256:f3cfc9d0dbf931d3db4685ec659b7ac68e2a578219da4aae65427886e649b06b"},"type":"cosign container image signature"},"optional":null}"#; let err = verification_key - .verify_signature(signature, &msg.as_bytes()) + .verify_signature(signature, msg.as_bytes()) .expect_err("Was expecting an error"); let found = match err { SigstoreError::Base64DecodeError(_) => true, @@ -344,7 +344,7 @@ JsB89BPhZYch0U0hKANx5TY+ncrm0s8bfJxxHoenAEFhwhuXeb4PqIrtoQ== let msg = r#"{"critical":{"identity":{"docker-reference":"registry-testing.svc.lan/busybox"},"image":{"docker-manifest-digest":"sha256:f3cfc9d0dbf931d3db4685ec659b7ac68e2a578219da4aae65427886e649b06b"},"type":"cosign container image signature"},"optional":null}"#; let err = verification_key - .verify_signature(signature, &msg.as_bytes()) + .verify_signature(signature, msg.as_bytes()) .expect_err("Was expecting an error"); let found = match err { SigstoreError::PublicKeyVerificationError => true, @@ -374,7 +374,7 @@ DwIDAQAB let msg = r#"{"critical":{"identity":{"docker-reference":"registry.suse.com/suse/sle-micro/5.0/toolbox"},"image":{"docker-manifest-digest":"sha256:356631f7603526a0af827741f5fe005acf19b7ef7705a34241a91c2d47a6db5e"},"type":"cosign container image signature"},"optional":{"creator":"OBS"}}"#; assert!(verification_key - .verify_signature(signature, &msg.as_bytes()) + .verify_signature(signature, msg.as_bytes()) .is_ok()); } } diff --git a/src/rekor/apis/entries_api.rs b/src/rekor/apis/entries_api.rs index db051c3deb..19e9b4652c 100644 --- a/src/rekor/apis/entries_api.rs +++ b/src/rekor/apis/entries_api.rs @@ -52,7 +52,7 @@ pub enum SearchLogQueryError { UnknownValue(serde_json::Value), } -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct LogEntries { entries: Vec, diff --git a/src/rekor/models/alpine.rs b/src/rekor/models/alpine.rs index 454d62c33f..a9fca265c8 100644 --- a/src/rekor/models/alpine.rs +++ b/src/rekor/models/alpine.rs @@ -11,7 +11,7 @@ use serde::{Deserialize, Serialize}; /// Alpine : Alpine package -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Alpine { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/alpine_all_of.rs b/src/rekor/models/alpine_all_of.rs index fccb589116..d64cb384cd 100644 --- a/src/rekor/models/alpine_all_of.rs +++ b/src/rekor/models/alpine_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct AlpineAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/consistency_proof.rs b/src/rekor/models/consistency_proof.rs index f0927c945d..f819675eed 100644 --- a/src/rekor/models/consistency_proof.rs +++ b/src/rekor/models/consistency_proof.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct ConsistencyProof { /// The hash value stored at the root of the merkle tree at the time the proof was generated #[serde(rename = "rootHash")] diff --git a/src/rekor/models/error.rs b/src/rekor/models/error.rs index ac587747fb..a146f63ebf 100644 --- a/src/rekor/models/error.rs +++ b/src/rekor/models/error.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Error { #[serde(rename = "code", skip_serializing_if = "Option::is_none")] pub code: Option, diff --git a/src/rekor/models/hashedrekord.rs b/src/rekor/models/hashedrekord.rs index 6319241294..83f96aad7b 100644 --- a/src/rekor/models/hashedrekord.rs +++ b/src/rekor/models/hashedrekord.rs @@ -13,7 +13,7 @@ use url::Url; /// Hashedrekord : Hashed Rekord object -#[derive(Clone, Debug, PartialEq, Serialize, Deserialize)] +#[derive(Clone, Debug, Eq, PartialEq, Serialize, Deserialize)] pub struct Hashedrekord { #[serde(rename = "kind")] pub kind: String, @@ -35,7 +35,7 @@ impl Hashedrekord { } /// Stores the Signature and Data struct -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Spec { signature: Signature, @@ -50,7 +50,7 @@ impl Spec { } /// Stores the signature format, signature of the artifact and the PublicKey struct -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Signature { format: String, @@ -69,7 +69,7 @@ impl Signature { } /// Stores the public key used to sign the artifact -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct PublicKey { content: String, @@ -81,7 +81,7 @@ impl PublicKey { } } -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Data { hash: Hash, @@ -94,7 +94,7 @@ impl Data { } } -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[allow(non_camel_case_types)] pub enum AlgorithmKind { sha256, @@ -102,7 +102,7 @@ pub enum AlgorithmKind { } /// Stores the algorithm used to hash the artifact and the value of the hash -#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Hash { algorithm: AlgorithmKind, diff --git a/src/rekor/models/hashedrekord_all_of.rs b/src/rekor/models/hashedrekord_all_of.rs index c1f73137ad..818cc9563f 100644 --- a/src/rekor/models/hashedrekord_all_of.rs +++ b/src/rekor/models/hashedrekord_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct HashedrekordAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/helm.rs b/src/rekor/models/helm.rs index 1584416967..64e682acad 100644 --- a/src/rekor/models/helm.rs +++ b/src/rekor/models/helm.rs @@ -12,7 +12,7 @@ use serde::{Deserialize, Serialize}; /// Helm : Helm chart -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Helm { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/helm_all_of.rs b/src/rekor/models/helm_all_of.rs index 223373caed..e925c02344 100644 --- a/src/rekor/models/helm_all_of.rs +++ b/src/rekor/models/helm_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct HelmAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/inactive_shard_log_info.rs b/src/rekor/models/inactive_shard_log_info.rs index 3054771778..f33bab9b4c 100644 --- a/src/rekor/models/inactive_shard_log_info.rs +++ b/src/rekor/models/inactive_shard_log_info.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct InactiveShardLogInfo { /// The current hash value stored at the root of the merkle tree #[serde(rename = "rootHash")] diff --git a/src/rekor/models/inclusion_proof.rs b/src/rekor/models/inclusion_proof.rs index 3dca24a4b4..76038dd510 100644 --- a/src/rekor/models/inclusion_proof.rs +++ b/src/rekor/models/inclusion_proof.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct InclusionProof { /// The index of the entry in the transparency log #[serde(rename = "logIndex")] diff --git a/src/rekor/models/intoto.rs b/src/rekor/models/intoto.rs index 85b526ed3b..31d6f2a6cb 100644 --- a/src/rekor/models/intoto.rs +++ b/src/rekor/models/intoto.rs @@ -12,7 +12,7 @@ use serde::{Deserialize, Serialize}; /// Intoto : Intoto object -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Intoto { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/intoto_all_of.rs b/src/rekor/models/intoto_all_of.rs index 8adc57c234..09548987f4 100644 --- a/src/rekor/models/intoto_all_of.rs +++ b/src/rekor/models/intoto_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct IntotoAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/jar.rs b/src/rekor/models/jar.rs index 266436fb9d..14132fd8aa 100644 --- a/src/rekor/models/jar.rs +++ b/src/rekor/models/jar.rs @@ -11,7 +11,7 @@ /// Jar : Java Archive (JAR) use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Jar { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/jar_all_of.rs b/src/rekor/models/jar_all_of.rs index 839e450425..7615dfd77f 100644 --- a/src/rekor/models/jar_all_of.rs +++ b/src/rekor/models/jar_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct JarAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/log_entry.rs b/src/rekor/models/log_entry.rs index 5134433e11..b817cbcba6 100644 --- a/src/rekor/models/log_entry.rs +++ b/src/rekor/models/log_entry.rs @@ -1,7 +1,7 @@ use serde::{Deserialize, Serialize}; /// Stores the response returned by Rekor after making a new entry -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, Eq, PartialEq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct LogEntry { uuid: String, @@ -14,7 +14,7 @@ pub struct LogEntry { verification: Verification, } -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Attestation { // This field is just a place holder @@ -24,7 +24,7 @@ pub struct Attestation { } /// Stores the signature over the artifact's logID, logIndex, body and integratedTime. -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Verification { #[serde(skip_serializing_if = "Option::is_none")] @@ -33,7 +33,7 @@ pub struct Verification { } /// Stores the signature over the artifact's logID, logIndex, body and integratedTime. -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct InclusionProof { hashes: Vec, diff --git a/src/rekor/models/log_info.rs b/src/rekor/models/log_info.rs index ed48a1620b..5b9e4e8e65 100644 --- a/src/rekor/models/log_info.rs +++ b/src/rekor/models/log_info.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct LogInfo { /// The current hash value stored at the root of the merkle tree #[serde(rename = "rootHash")] diff --git a/src/rekor/models/proposed_entry.rs b/src/rekor/models/proposed_entry.rs index 2b6a6088fb..a395d49209 100644 --- a/src/rekor/models/proposed_entry.rs +++ b/src/rekor/models/proposed_entry.rs @@ -9,7 +9,7 @@ */ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Serialize, Deserialize)] +#[derive(Clone, Debug, Eq, PartialEq, Serialize, Deserialize)] #[serde(tag = "kind")] pub enum ProposedEntry { #[serde(rename = "alpine")] diff --git a/src/rekor/models/rekord.rs b/src/rekor/models/rekord.rs index d64baf2f53..a5bd4db98b 100644 --- a/src/rekor/models/rekord.rs +++ b/src/rekor/models/rekord.rs @@ -11,7 +11,7 @@ /// Rekord : Rekord object use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Rekord { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/rekord_all_of.rs b/src/rekor/models/rekord_all_of.rs index a31f882f43..c3901171a9 100644 --- a/src/rekor/models/rekord_all_of.rs +++ b/src/rekor/models/rekord_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct RekordAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/rfc3161.rs b/src/rekor/models/rfc3161.rs index 369ed835e4..4c8617339e 100644 --- a/src/rekor/models/rfc3161.rs +++ b/src/rekor/models/rfc3161.rs @@ -11,7 +11,7 @@ /// Rfc3161 : RFC3161 Timestamp use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Rfc3161 { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/rfc3161_all_of.rs b/src/rekor/models/rfc3161_all_of.rs index 289d88c863..98880eb19d 100644 --- a/src/rekor/models/rfc3161_all_of.rs +++ b/src/rekor/models/rfc3161_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Rfc3161AllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/rpm.rs b/src/rekor/models/rpm.rs index e5f4b77e19..a7a9998ed3 100644 --- a/src/rekor/models/rpm.rs +++ b/src/rekor/models/rpm.rs @@ -12,7 +12,7 @@ use serde::{Deserialize, Serialize}; /// Rpm : RPM package -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Rpm { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/rpm_all_of.rs b/src/rekor/models/rpm_all_of.rs index 027283481e..fc7499dea1 100644 --- a/src/rekor/models/rpm_all_of.rs +++ b/src/rekor/models/rpm_all_of.rs @@ -9,7 +9,7 @@ */ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct RpmAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/rekor/models/search_index.rs b/src/rekor/models/search_index.rs index c70225a33b..63586fddac 100644 --- a/src/rekor/models/search_index.rs +++ b/src/rekor/models/search_index.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct SearchIndex { #[serde(rename = "email", skip_serializing_if = "Option::is_none")] pub email: Option, diff --git a/src/rekor/models/search_index_public_key.rs b/src/rekor/models/search_index_public_key.rs index 58a7f38310..7f2fb3a038 100644 --- a/src/rekor/models/search_index_public_key.rs +++ b/src/rekor/models/search_index_public_key.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct SearchIndexPublicKey { #[serde(rename = "format")] pub format: Format, diff --git a/src/rekor/models/search_log_query.rs b/src/rekor/models/search_log_query.rs index 0c44a1d016..55fac17034 100644 --- a/src/rekor/models/search_log_query.rs +++ b/src/rekor/models/search_log_query.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, Eq, PartialEq, Default, Serialize, Deserialize)] pub struct SearchLogQuery { #[serde(rename = "entryUUIDs", skip_serializing_if = "Option::is_none")] pub entry_uuids: Option>, diff --git a/src/rekor/models/tuf.rs b/src/rekor/models/tuf.rs index ce3a091cc7..a512149cfd 100644 --- a/src/rekor/models/tuf.rs +++ b/src/rekor/models/tuf.rs @@ -12,7 +12,7 @@ use serde::{Deserialize, Serialize}; /// Tuf : TUF metadata -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct Tuf { #[serde(rename = "kind")] pub kind: String, diff --git a/src/rekor/models/tuf_all_of.rs b/src/rekor/models/tuf_all_of.rs index 875573e378..7ec6e5acdf 100644 --- a/src/rekor/models/tuf_all_of.rs +++ b/src/rekor/models/tuf_all_of.rs @@ -10,7 +10,7 @@ use serde::{Deserialize, Serialize}; -#[derive(Clone, Debug, PartialEq, Default, Serialize, Deserialize)] +#[derive(Clone, Debug, PartialEq, Eq, Default, Serialize, Deserialize)] pub struct TufAllOf { #[serde(rename = "apiVersion")] pub api_version: String, diff --git a/src/tuf/repository_helper.rs b/src/tuf/repository_helper.rs index d5e36a3a8b..917237145c 100644 --- a/src/tuf/repository_helper.rs +++ b/src/tuf/repository_helper.rs @@ -278,7 +278,7 @@ mod tests { .join("targets") .join(filename), ) - .expect(format!("cannot read {} from test data", filename).as_str()); + .unwrap_or_else(|_| panic!("cannot read {} from test data", filename)); crate::registry::Certificate { data, encoding: crate::registry::CertificateEncoding::Pem, @@ -329,7 +329,7 @@ mod tests { .join("targets") .join(filename), ) - .expect(format!("cannot read {} from test data", filename).as_str()); + .unwrap_or_else(|_| panic!("cannot read {} from test data", filename)); crate::registry::Certificate { data, encoding: crate::registry::CertificateEncoding::Pem, @@ -358,7 +358,7 @@ mod tests { let cache_dir = TempDir::new().expect("Cannot create temp cache dir"); // put some outdated files inside of the cache - for filename in vec!["fulcio.crt.pem", "fulcio_v1.crt.pem"] { + for filename in &["fulcio.crt.pem", "fulcio_v1.crt.pem"] { fs::write(cache_dir.path().join(filename), b"fake fulcio") .expect("Cannot write file to cache dir"); } @@ -386,7 +386,7 @@ mod tests { .join("targets") .join(filename), ) - .expect(format!("cannot read {} from test data", filename).as_str()); + .unwrap_or_else(|_| panic!("cannot read {} from test data", filename)); crate::registry::Certificate { data, encoding: crate::registry::CertificateEncoding::Pem,