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

update NYTimes/gziphandler fixes #1059 #1084

Merged

Conversation

JamesKyburz
Copy link
Contributor

Updating gziphandler to use commit 6710af535839f57c687b62c4c23d649f9545d885 fixes issue #1059.

@emilevauge emilevauge force-pushed the feature/fix-gzip-for-websockets branch from f2a9c14 to d516cbf Compare February 2, 2017 16:56
@emilevauge emilevauge added this to the 1.2 milestone Feb 2, 2017
Copy link
Member

@emilevauge emilevauge left a comment

Choose a reason for hiding this comment

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

Thanks @JamesKyburz
LGTM

@emilevauge
Copy link
Member

/cc @containous/traefik

Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐸

@vdemeester vdemeester merged commit 89ec25f into traefik:master Feb 4, 2017
@ldez ldez added the kind/bug/fix a bug fix label Apr 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug/fix a bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants