Wrong Content-Type header

Created on 26 June 2023, over 1 year ago

Updated 4 October 2023, about 1 year ago

Problem/Motivation

AVIF images returned by this module have the wrong Content-Type header, they keep the original type of the image (image/png, for example).

Steps to reproduce

Browse to an AVIF URL and inspect the response headers.

Proposed resolution

ImageStyleDownloadController::avifResponse() should use array_merge() instead of += to merge headers (because += does not override existing keys).

🐛 Bug report

Status

Closed: outdated

Version

1.0

Component

Code

Created by

🇫🇷France prudloff Lille

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @prudloff
@prudloff opened merge request.
Status changed to Needs review over 1 year ago4:22pm 26 June 2023
Comment over 1 year ago →
🇫🇷France prudloff Lille
Status changed to Closed: outdated about 1 year ago3:18am 4 October 2023
Comment about 1 year ago →
🇦🇺Australia mstrelan
This is no longer applicable since 📌 Simplify ImageStyleDownloadController by decorating core Needs review .

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024