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 possibility to configure Jetty's minRequestDataRate #2184

Merged
merged 2 commits into from Oct 25, 2017

Conversation

Projects
None yet
4 participants
@patrox
Contributor

patrox commented Oct 24, 2017

Resolves #2181.

@coveralls

This comment has been minimized.

coveralls commented Oct 24, 2017

Coverage Status

Coverage increased (+0.01%) to 85.484% when pulling eec0a47 on patrox:issue_2181_jetty_minRequestDataRate into d3a7b57 on dropwizard:master.

@joschi joschi self-assigned this Oct 25, 2017

@joschi

LGTM. 👍

Please add the new setting to the documentation, too.

@patrox patrox force-pushed the patrox:issue_2181_jetty_minRequestDataRate branch from abaa1d5 to f7f9872 Oct 25, 2017

@coveralls

This comment has been minimized.

coveralls commented Oct 25, 2017

Coverage Status

Coverage increased (+0.01%) to 85.484% when pulling f7f9872 on patrox:issue_2181_jetty_minRequestDataRate into d3a7b57 on dropwizard:master.

@joschi

joschi approved these changes Oct 25, 2017

@coveralls

This comment has been minimized.

coveralls commented Oct 25, 2017

Coverage Status

Coverage increased (+0.01%) to 85.484% when pulling f7f9872 on patrox:issue_2181_jetty_minRequestDataRate into d3a7b57 on dropwizard:master.

@joschi joschi merged commit 9b35878 into dropwizard:master Oct 25, 2017

4 of 5 checks passed

codeclimate 2 issues to fix
Details
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.01%) to 85.484%
Details

@joschi joschi added this to the 2.0.0 milestone Oct 25, 2017

@joschi joschi added the improvement label Oct 25, 2017

@arteam arteam modified the milestones: 2.0.0, 1.2.1 Oct 26, 2017

sankate pushed a commit to sankate/dropwizard that referenced this pull request Nov 21, 2017

@nickbabcock nickbabcock referenced this pull request Sep 14, 2018

Merged

Update to jetty 9.4.12 #2490

1 of 2 tasks complete

aaanders added a commit to aaanders/dropwizard that referenced this pull request Sep 20, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment