use clone() insteaf of to_string() when accessing issuer

Signed-off-by: Anton Engelhardt <[email protected]>
antonengelhardt · May 6, 2024 · ce61e4a · ce61e4a
1 parent ada5db2
commit ce61e4a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/discovery.rs b/src/discovery.rs
@@ -409,7 +409,7 @@ impl Context for OidcDiscovery {
                             open_id_config: Arc::new(OpenIdConfig {
                                 auth_endpoint: auth_endpoint.clone(),
                                 token_endpoint: token_endpoint.clone(),
-                                issuer: issuer.to_string(),
+                                issuer: issuer.clone(),
                                 public_keys: keys,
                             }),
                             plugin_config: plugin_config.clone(),

diff --git a/src/lib.rs b/src/lib.rs
@@ -379,7 +379,7 @@ impl ConfiguredOidc {
         // Define allowed issuers and audiences
         let mut allowed_issuers = HashSet::new();
         // remove last slash from issuer url
-        allowed_issuers.insert(self.open_id_config.issuer.to_string());
+        allowed_issuers.insert(self.open_id_config.issuer.clone());
         let mut allowed_audiences = HashSet::new();
         allowed_audiences.insert(self.plugin_config.audience.to_string());