Let JwtAuthorizer::new set the RSA pubkey parse error message

Allow only a subset of `JwtAuthConfig` fields to be set in config file,
as we allow any members of the configuration tables to not be set in
the file, and instead be set by the corresponding environment variable.

Author: tankyleo

rust/auth-impls/src/jwt.rs

@@ -34,8 +34,8 @@ const BEARER_PREFIX: &str = "Bearer ";
 impl JWTAuthorizer {
 	/// Creates a new instance of [`JWTAuthorizer`], fails on failure to parse the PEM formatted RSA public key
 	pub async fn new(rsa_pem: &str) -> Result<Self, String> {
-		let jwt_issuer_key =
-			DecodingKey::from_rsa_pem(rsa_pem.as_bytes()).map_err(|e| e.to_string())?;
+		let jwt_issuer_key = DecodingKey::from_rsa_pem(rsa_pem.as_bytes())
+			.map_err(|e| format!("Failed to parse the PEM formatted RSA public key: {}", e))?;
 		Ok(Self { jwt_issuer_key })
 	}
 }

rust/server/src/main.rs

@@ -82,15 +82,15 @@ fn main() {
 					std::process::exit(-1);
 				},
 			};
-			let rsa_pem = rsa_pem_env.or(jwt_auth_config.map(|config| config.rsa_pem));
+			let rsa_pem = rsa_pem_env.or(jwt_auth_config.and_then(|config| config.rsa_pem));
 			if let Some(pem) = rsa_pem {
 				authorizer = match JWTAuthorizer::new(pem.as_str()).await {
 					Ok(auth) => {
 						println!("Configured JWT authorizer with RSA public key");
 						Some(Arc::new(auth))
 					},
 					Err(e) => {
-						println!("Failed to parse the PEM formatted RSA public key: {}", e);
+						println!("Failed to configure JWT authorizer: {}", e);
 						std::process::exit(-1);
 					},
 				};

rust/server/src/util/config.rs

@@ -15,7 +15,7 @@ pub(crate) struct ServerConfig {
 
 #[derive(Deserialize)]
 pub(crate) struct JwtAuthConfig {
-	pub(crate) rsa_pem: String,
+	pub(crate) rsa_pem: Option<String>,
 }
 
 #[derive(Deserialize)]