Skip to content

do not compress media types#3075

Merged
robjtede merged 17 commits into
masterfrom
no-compress-media
Jul 19, 2023
Merged

do not compress media types#3075
robjtede merged 17 commits into
masterfrom
no-compress-media

Conversation

@robjtede

@robjtede robjtede commented Jul 19, 2023

Copy link
Copy Markdown
Member

PR Type

Fix

PR Checklist

  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • A changelog entry has been made for the appropriate packages.
  • Format code with the latest stable rustfmt.
  • (Team) Label with affected crates and semver status.

Overview

Split out from #2996

Image and video types are mostly compressed already, so do not compress them again.

Fixes #2981

@robjtede robjtede added B-semver-minor A-web project: actix-web labels Jul 19, 2023
@robjtede robjtede added this to the actix-web v4.4 milestone Jul 19, 2023
@robjtede robjtede enabled auto-merge (squash) July 19, 2023 19:24
@robjtede robjtede disabled auto-merge July 19, 2023 19:24
@robjtede robjtede merged commit db99da5 into master Jul 19, 2023
@robjtede robjtede deleted the no-compress-media branch July 19, 2023 19:24
@Degubi Degubi mentioned this pull request Oct 16, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-web project: actix-web B-semver-minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compress middleware should not compress image/* content types

2 participants