Rename invalid CSR signature error variant

Author: djc

rcgen/src/csr.rs

@@ -92,7 +92,8 @@ impl CertificateSigningRequestParams {
 	/// - `Extended Key Usage` (see [`ExtendedKeyUsagePurpose`])
 	///
 	/// On encountering other extensions, this function will return [`Error::UnsupportedExtension`].
-	/// If the request's signature is invalid, it will return [`Error::InvalidRequestSignature`].
+	/// If the request's signature is invalid, it will return
+	/// [`Error::InvalidCertificationRequestSignature`].
 	///
 	/// The [`PemObject`] trait is often used to obtain a [`CertificateSigningRequestDer`] from
 	/// PEM input. If you already have a byte slice containing DER, it can trivially be converted
@@ -107,7 +108,7 @@ impl CertificateSigningRequestParams {
 			.map_err(|_| Error::CouldNotParseCertificationRequest)?
 			.1;
 		csr.verify_signature()
-			.map_err(|_| Error::InvalidRequestSignature)?;
+			.map_err(|_| Error::InvalidCertificationRequestSignature)?;
 		let alg_oid = csr
 			.signature_algorithm
 			.algorithm

rcgen/src/error.rs

@@ -12,7 +12,7 @@ pub enum Error {
 	CouldNotParseKeyPair,
 	/// The CSR signature is invalid
 	#[cfg(feature = "x509-parser")]
-	InvalidRequestSignature,
+	InvalidCertificationRequestSignature,
 	#[cfg(feature = "x509-parser")]
 	/// Invalid subject alternative name type
 	InvalidNameType,
@@ -65,7 +65,7 @@ impl fmt::Display for Error {
 			)?,
 			CouldNotParseKeyPair => write!(f, "Could not parse key pair")?,
 			#[cfg(feature = "x509-parser")]
-			InvalidRequestSignature => write!(f, "Invalid CSR signature")?,
+			InvalidCertificationRequestSignature => write!(f, "Invalid CSR signature")?,
 			#[cfg(feature = "x509-parser")]
 			InvalidNameType => write!(f, "Invalid subject alternative name type")?,
 			InvalidAsn1String(e) => write!(f, "{e}")?,