dotnet/aspnetcore

Compression: Add threshold for compression

Open

#5,994 opened on Dec 5, 2016

View on GitHub
 (11 comments) (2 reactions) (0 assignees)C# (10,653 forks)batch import
affected-fewarea-middlewareenhancementfeature-response-compressionhelp wantedseverity-minor

Repository metrics

Stars
 (37,933 stars)
PR merge metrics
 (Avg merge 16d 9h) (258 merged PRs in 30d)

Description

There are instances where compression is a detriment by comparison to the benefit. Where the time to compress and gzip payload is higher than the original payload.

ngnix defaults to all content with the gzip_min_length setting used to set the minimum length of a response to that will be gzipped.

expressjs compression module is 1k by default, uses threshold option to set globally.

Contributor guide

Research direction
Add a configurable threshold option to the compression middleware to skip compressing responses below a certain size.
Tech stack
csharp
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C#ASP.NET Core
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60