Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

do not compress media types #3075

Merged
merged 17 commits into from
Jul 19, 2023
Merged

do not compress media types #3075

merged 17 commits into from
Jul 19, 2023

Conversation

robjtede
Copy link
Member

@robjtede robjtede commented Jul 19, 2023

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
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