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

Bump waitress from 2.1.1 to 2.1.2 #678

Merged
merged 4 commits into from
Apr 6, 2023
Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jun 2, 2022

Bumps waitress from 2.1.1 to 2.1.2.

Changelog

Sourced from waitress's changelog.

2.1.2

Bugfix


- When expose_tracebacks is enabled waitress would fail to properly encode
  unicode thereby causing another error during error handling. See
  https://github.com/Pylons/waitress/pull/378
  • Header length checking had a calculation that was done incorrectly when the data was received across multple socket reads. This calculation has been corrected, and no longer will Waitress send back a 413 Request Entity Too Large. See Pylons/waitress#376

Security Bugfix

  • in 2.1.0 a new feature was introduced that allowed the WSGI thread to start sending data to the socket. However this introduced a race condition whereby a socket may be closed in the sending thread while the main thread is about to call select() therey causing the entire application to be taken down. Waitress will no longer close the socket in the WSGI thread, instead waking up the main thread to cleanup. See Pylons/waitress#377
Commits
  • 0aa4879 Remove change of default for clear_untrusted_proxy_headers
  • 2784628 Revert "Merge pull request #370 from Yourun-proger/del_warnings"
  • 479df63 Prep 2.1.2
  • 4f6789b Merge pull request #377 from Pylons/bugfix/select-closed-socket-race
  • 1952050 Merge pull request #379 from Pylons/enhancement/pyupgrade-3.7
  • 8f5b473 pyupgrade 3.7
  • c7a3d7e Only close socket in the main thread
  • 7c3739b Merge pull request #376 from Pylons/bugfix/header-calculation
  • 3066fdd Merge pull request #378 from Pylons/bugfix/expose_tracebacks-encode-error
  • 4467d76 Fix tests to assume body is bytes
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

@dependabot dependabot bot requested a review from a team as a code owner June 2, 2022 20:51
@dependabot dependabot bot requested review from nsheaps and KTAtkinson June 2, 2022 20:51
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Jun 2, 2022
Copy link
Contributor

@KTAtkinson KTAtkinson left a comment

Choose a reason for hiding this comment

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

Considering there are no security patches or needed bugfixes in this version,a 94% compatibility rating seems like a little more risk than I would like. Thoughts @nsheaps?

Copy link
Contributor

@nsheaps nsheaps left a comment

Choose a reason for hiding this comment

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

@KTAtkinson I don't think the change is that drastic considering the actual code changes, but I'm not clear on how to ensure that this works properly for all of our use cases other than merging and having people try it out (beyond the tests that currently exist). We could upgrade some of the services we own and test it out there but based on our past upgrade experiences I don't think that's sufficient unfortunately.

Bumps [waitress](https://github.com/Pylons/waitress) from 2.1.1 to 2.1.2.
- [Release notes](https://github.com/Pylons/waitress/releases)
- [Changelog](https://github.com/Pylons/waitress/blob/v2.1.2/CHANGES.txt)
- [Commits](Pylons/waitress@v2.1.1...v2.1.2)

---
updated-dependencies:
- dependency-name: waitress
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@nsheaps nsheaps force-pushed the dependabot/pip/waitress-2.1.2 branch from df2632c to 0cc7bd5 Compare July 26, 2022 21:45
@nsheaps
Copy link
Contributor

nsheaps commented Dec 8, 2022

Our testing methodology has improved to the point where we should be able to validate this

Copy link
Contributor

@KTAtkinson KTAtkinson left a comment

Choose a reason for hiding this comment

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

Should go out with next release.

@nsheaps
Copy link
Contributor

nsheaps commented Dec 15, 2022

Thanks Katie!

🏃

@KTAtkinson KTAtkinson added the v2.6 Pull requests to be included in v2.6 label Apr 4, 2023
@KTAtkinson KTAtkinson merged commit 2c156c5 into develop Apr 6, 2023
@KTAtkinson KTAtkinson deleted the dependabot/pip/waitress-2.1.2 branch April 6, 2023 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file v2.6 Pull requests to be included in v2.6
Development

Successfully merging this pull request may close these issues.

2 participants