1980239 - moz-cached-ohttp channel should set contentType on itself when pulling resources from the cache

Status: RESOLVED FIXED

(Firefox :: New Tab Page, task)

Description

[Mike Conley (:mconley) (:⚙️) (PTO June 8 - 12)]

At load time, when getting (for example, an SVG image) over OHTTP, we correctly set the contentType on the channel by reading the content type header from the nsIObliviousHttpChannel.

When, however, we pull the image from the cache, we do no such thing. The existing implementation has not been caching the content type (which it should), and when we pull from the cache, we should update the content type on the channel to match it.

Comment 1

[Mike Conley (:mconley) (:⚙️) (PTO June 8 - 12)]

Attachment: Bug 1980239 - Include the content type when storing / retrieving cache entries via moz-cached-ohttp. r?kershaw!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/3c8ab5f89cbe
https://hg.mozilla.org/integration/autoland/rev/cd17d932f90c
Include the content type when storing / retrieving cache entries via moz-cached-ohttp. r=kershaw

Comment 3

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/cd17d932f90c