use Option.iter for eTag and lastModified values at Precondition.fs, …

…due to ionide-analyzers HandleOptionGracefullyAnalyzer (#613)
giraffe-fsharp · Sep 5, 2024 · 1edaf3f · 1edaf3f
1 parent 79910c9
commit 1edaf3f
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/Giraffe/Preconditional.fs b/src/Giraffe/Preconditional.fs
@@ -150,12 +150,12 @@ type PreconditionExtensions() =
             | ConditionFailed -> ConditionFailed
             | ResourceNotModified -> ResourceNotModified
 
-        // Set ETag and Last-Modified in the response
-        if eTag.IsSome then
-            responseHeaders.ETag <- eTag.Value
+        // Set ETag in the response
+        eTag |> Option.iter (fun eTagValue -> responseHeaders.ETag <- eTagValue)
 
-        if lastModified.IsSome then
-            responseHeaders.LastModified <- Nullable(lastModified.Value.CutOffMs())
+        // Set Last-Modified in the response
+        lastModified
+        |> Option.iter (fun lastModifiedValue -> responseHeaders.LastModified <- Nullable(lastModifiedValue.CutOffMs()))
 
         // Validate headers in correct precedence
         // RFC: https://tools.ietf.org/html/rfc7232#section-6