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

Add HTTP Compression #61

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add HTTP Compression #61

wants to merge 4 commits into from

Conversation

imclint21
Copy link
Member

Hi,

This is a proposition for adding a compression layer to the HTTP server, not the object compression into the store.

For now warp does not have released this new compression feature to crates, so I think we can merge this PR when it will be the case!

Kind regards

Clint.Network added 3 commits March 3, 2020 17:07
Using direct repository for now, compression feature is not pushed to crates
@imclint21 imclint21 added enhancement New feature or request help wanted Extra attention is needed core Core development proposition labels Mar 15, 2020
@imclint21 imclint21 added this to the alpha milestone Mar 15, 2020
@imclint21 imclint21 requested a review from shuni64 March 15, 2020 21:59
@imclint21 imclint21 self-assigned this Mar 15, 2020
Copy link
Contributor

@shuni64 shuni64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good so far, although it contains unrelated commits.
Please only request a review when the feature is already done though, there's not much to review right now.

@imclint21
Copy link
Member Author

Alright, no problem, it's just when I fill the gh sidebar

Cargo.toml Outdated Show resolved Hide resolved
@imclint21 imclint21 modified the milestones: alpha, beta Apr 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core development enhancement New feature or request help wanted Extra attention is needed proposition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants