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

updates jet version for improved trailers support #712

Merged
merged 2 commits into from Apr 26, 2019

Conversation

Projects
None yet
2 participants
@shamsimam
Copy link
Contributor

commented Apr 25, 2019

Changes proposed in this PR

Why are we making these changes?

We want to avoid sending empty trailer Header frames for HTTP/2 requests when we know that there are no trailers.

@shamsimam shamsimam self-assigned this Apr 25, 2019

@shamsimam shamsimam requested a review from sradack Apr 25, 2019

@shamsimam shamsimam force-pushed the http2-late-binding-trailers branch from 4980db2 to 3e7e0c3 Apr 26, 2019

@shamsimam

This comment has been minimized.

Copy link
Contributor Author

commented Apr 26, 2019

@sradack this is good to merge.

@sradack sradack merged commit 717a499 into master Apr 26, 2019

2 checks passed

Mergeable Mergeable Run has been Completed!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@sradack sradack deleted the http2-late-binding-trailers branch Apr 26, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.