Merge pull request #77 from sebadob/no-session-ip-unwrap

no unwrap on existing session ip
sebadob · Oct 11, 2023 · 6dfd0f4 · 6dfd0f4
2 parents 5fb7af4 + c5c8837
commit 6dfd0f4
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/rauthy-models/src/entity/sessions.rs b/rauthy-models/src/entity/sessions.rs
@@ -440,11 +440,14 @@ impl Session {
             if (self.state == SessionState::Open || self.state == SessionState::Auth)
                 && self.remote_ip.as_ref() != Some(&ip)
             {
+                let session_ip = self
+                    .remote_ip
+                    .as_ref()
+                    .map(|ip| ip.as_str())
+                    .unwrap_or("UNKNOWN");
                 warn!(
                     "Invalid access for session {} / {} with different IP: {}",
-                    self.id,
-                    self.remote_ip.as_ref().unwrap(),
-                    ip,
+                    self.id, session_ip, ip,
                 );
                 return false;
             }