chore (api): remove big comment in favor of using a link that points to a giraffe discussion

Author: 64J0

fsharp-api/Router/Prometheus.middleware.fs

@@ -25,28 +25,8 @@ let requestCounter: HttpHandler =
 
             requestCounter.WithLabels(endpoint, method, statusCode).Inc()
 
-            // Skip the remaining pipeline
-            //
-            // WARNING!!!!
-            //
-            // We need to return `Some ctx` instead of `None` here, otherwise, when
-            // reaching the **handleResult** (https://github.com/giraffe-fsharp/Giraffe/blob/master/src/Giraffe/EndpointRouting.fs#L101-L104)
-            // Giraffe will try to set the response status code for a request that
-            // was already finished. This will throw some exception that one can
-            // finds at the server logs.
-            //
-            // Something that I still don't fully understand is why this exception
-            // related to the status code change does not happen for all the cases.
-            // When testing it was only happening when the /api/prediction endpoint
-            // fails (i.e., throws an exception itself).
-            //
-            // To test this error locally, just change this "return Some ctx" to
-            // "return None", start the API and use the /api/prediction endpoint with
-            // an invalid payload, like:
-            //
-            // curl -X POST -H "Accept: application/json" -d '{"id":"1", "crimesPerCapta":0.01}' localhost:8088/api/prediction
-            //
-            // P.S.: Notice that the id is a string instead of an integer ^.
+            // Check this discussion for why we can't use return None:
+            // - https://github.com/giraffe-fsharp/Giraffe/discussions/659
             return Some ctx
         }