Review feedback

src/Sentry.AspNetCore/RequestDecompression/RequestDecompressionMiddleware.cs

@@ -100,7 +100,7 @@ private void HandleException(Exception e)
         const string description =
             "An exception was captured and then re-thrown, when attempting to decompress the request body." +
             "The web server likely returned a 5xx error code as a result of this exception.";
-        e.SetSentryMechanism("RequestDecompressionMiddleware", description, handled: false);
+        e.SetSentryMechanism(nameof(RequestDecompressionMiddleware), description, handled: false);
         _hub.CaptureException(e);
         ExceptionDispatchInfo.Capture(e).Throw();
     }

src/Sentry.AspNetCore/SentryAspNetCoreOptions.cs

@@ -122,15 +122,15 @@ internal void SetEnvironment(IWebHostEnvironment hostingEnvironment)
 
             if (hostingEnvironment.IsProduction())
             {
-                Environment = Sentry.Internal.Constants.ProductionEnvironmentSetting;
+                Environment = Internal.Constants.ProductionEnvironmentSetting;
             }
             else if (hostingEnvironment.IsStaging())
             {
-                Environment = Sentry.Internal.Constants.StagingEnvironmentSetting;
+                Environment = Internal.Constants.StagingEnvironmentSetting;
             }
             else if (hostingEnvironment.IsDevelopment())
             {
-                Environment = Sentry.Internal.Constants.DevelopmentEnvironmentSetting;
+                Environment = Internal.Constants.DevelopmentEnvironmentSetting;
             }
             else
             {

src/Sentry/Extensibility/BaseRequestPayloadExtractor.cs

@@ -23,24 +23,22 @@ public abstract class BaseRequestPayloadExtractor : IRequestPayloadExtractor
             return null;
         }
 
-        // When RequestDecompression is enabled, the RequestDecompressionMiddleware will store a SizeLimitedStream
-        // in the request body after decompression. Seek operations throw an exception, but we can still read the stream
-        var originalPosition = request.Body.CanSeek ? request.Body.Position : 0;
-        try
+        if (!request.Body.CanSeek)
         {
-            if (request.Body.CanSeek)
-            {
-                request.Body.Position = 0;
-            }
+            // When RequestDecompression is enabled, the RequestDecompressionMiddleware will store a SizeLimitedStream
+            // in the request body after decompression. Seek operations throw an exception, but we can still read the stream
+            return DoExtractPayLoad(request);
+        }
 
+        var  originalPosition = request.Body.Position;
+        try
+        {
+            request.Body.Position = 0;
             return DoExtractPayLoad(request);
         }
         finally
         {
-            if (request.Body.CanSeek)
-            {
-                request.Body.Position = originalPosition;
-            }
+            request.Body.Position = originalPosition;
         }
     }