SixLabors / ImageSharp

:camera: A modern, cross-platform, 2D Graphics library for .NET
https://sixlabors.com/products/imagesharp/
Other
7.46k stars 851 forks source link

WebP and TIFF encoders produce invalid images when writing to HttpContext.Response.Body #2806

Closed csboling closed 5 days ago

csboling commented 2 months ago

Prerequisites

ImageSharp version

3.1.5

Other ImageSharp packages and versions

N/A

Environment (Operating system, version and so on)

Windows 10 19044.2604, 64-bit

.NET Framework version

.NET 8

Description

Encoding an Image as WebP to a Microsoft.AspNetCore.Http.HttpContext (a non-seekable Stream) produces an image that cannot be displayed in the browser (Firefox 129.0.2, Chrome 128.0.6613.114) and has a different length and contents from the same Image written to a file. Writing an Image as TIFF in the same way produces a file which cannot be displayed in the Windows image viewer and contains no valid IFDs. All other formats currently supported by ImageSharp produce valid images when written in this way. Tested with both Image<Rgba32> and Image<Rgb24>. I would guess it affects other non-seekable Streams with these formats as well.

Steps to Reproduce

Here is a code sample reproducing the problem. A complete project can be found here: https://github.com/csboling/imagesharp-webp-http

using SixLabors.ImageSharp;
using SixLabors.ImageSharp.PixelFormats;

var builder = WebApplication.CreateBuilder(args);
var app = builder.Build();

app.MapGet(
    "/download.webp",
    async ctx =>
    {
        using (Image<Rgba32> image = new(1, 1))
        {
            ctx.Response.ContentType = "image/webp";
            await image.SaveAsWebpAsync(ctx.Response.Body);
        }
    });

app.Run();

Images

This ZIP contains example 1x1 pixel RGBA32 images encoded to WebP and TIFF to an HttpContext.Response, then downloaded with Chrome (invalid.webp, invalid.tif) as well as images encoded directly to a file (valid.webp, valid.tif).

examples.zip

JimBobSquarePants commented 2 months ago

This is odd. We explicitly wrap non seekable streams https://github.com/SixLabors/ImageSharp/blob/release/3.1.x/src/ImageSharp/Formats/ImageEncoder.cs

SamsidParty commented 2 months ago

I can reproduce this issue when writing to a Zip Output Stream, examining the webp file in a hex editor shows that the header doesn't contain a length value. Using version 2.1.9, it works fine.

JimBobSquarePants commented 2 months ago

Ok. Thanks for investigating. I’m on holiday in Europe currently don’t have access to a laptop just now so will need someone to debug the issue.

JimBobSquarePants commented 1 month ago

I've found a few issues with the current ChunkedMemoryStream implementation. Rewriting it now.

JimBobSquarePants commented 5 days ago

Fixed via #2828