Revert changes to BaseRequestPayloadExtractor

[jamescrosswell]

Resolves #4353

• Request.Body can only be read once when AddRequestDecompression is used #4353

Explanation

It doesn't look like the changes to BaseRequestPayloadExtractor were ever required. When I put a breakpoint here I see the stream is of type FileBufferingReadStream (indicating the EnableBuffering middleware is doing what it's supposed to).

Sanity check

There are already various unit tests for this but , as a sanity check, you can post a compressed body to the following program with something like:

echo -n 'your message here' | gzip | curl -X POST http://localhost:8080/message \
  --data-binary @- \
  -H "Content-Encoding: gzip" \
  -H "Content-Type: text/plain"

using Sentry.Extensibility;

var builder = WebApplication.CreateBuilder(args);

builder.WebHost.UseSentry(options =>
{
    // Log debug information about the Sentry SDK
    options.MaxRequestBodySize = RequestSize.Always;
    options.SendDefaultPii = true;
});

builder.Services.AddRequestDecompression();

var app = builder.Build();

app.UseRequestDecompression();

app.MapPost("message", _ =>
{
    // This is an example of a POST endpoint that captures the request body
    // and sends it to Sentry.
    SentrySdk.CaptureMessage("Hello from a POST endpoint!");

    return Task.CompletedTask;
});

[Flash0ver]

🤘