Repository navigation
chore(deps): bump the pip group across 1 directory with 2 updates - #344
dependabot[bot] wants to merge 1 commit into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
LGTM — straightforward dependency version bump.
What was reviewed: confirmed the diff touches only template/requirements.txt, bumping tornado 6.5.8→6.5.9 and urllib3 2.7.0→2.8.0 with no other lines changed. Both are minor/patch releases of actively maintained libraries (urllib3 2.8.0 notably includes security fixes), and no other code in the repo references pinned versions that would need updating.
Extended reasoning...
Single-file, two-line diff in template/requirements.txt bumping tornado and urllib3 patch versions via dependabot; no application code, auth, or data-handling logic is touched. Verified via git diff that nothing else changed. Low risk, mechanical dependency bump with upstream security fixes in urllib3, so safe to approve without further human review.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f587875d0e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| textblob==0.19.0 | ||
| tornado==6.5.8 | ||
| urllib3==2.7.0 | ||
| tornado==6.5.9 |
There was a problem hiding this comment.
Pin the Jupyter-compatible Tornado patch release
When a Jupyter FileFindHandler-backed resource such as /static/... is requested, Tornado 6.5.9 raises an AttributeError and returns HTTP 500 because Jupyter's handler is incompatible with the newly added allowed_symlink_directory state. This template pins and launches jupyter-server, so those routes become unusable; the Tornado 6.5.10 release notes explicitly identify the Jupyter regression and its fix, and 6.5.10 was already available before this commit.
Useful? React with 👍 / 👎.
Bumps the pip group with 2 updates in the /template directory: [tornado](https://github.com/tornadoweb/tornado) and [urllib3](https://github.com/urllib3/urllib3). Updates `tornado` from 6.5.8 to 6.5.9 - [Changelog](https://github.com/tornadoweb/tornado/blob/master/docs/releases.rst) - [Commits](tornadoweb/tornado@v6.5.8...v6.5.9) Updates `urllib3` from 2.7.0 to 2.8.0 - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](urllib3/urllib3@2.7.0...2.8.0) --- updated-dependencies: - dependency-name: tornado dependency-version: 6.5.9 dependency-type: direct:production - dependency-name: urllib3 dependency-version: 2.8.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
f587875 to
9c58735
Compare
Bumps the pip group with 2 updates in the /template directory: tornado and urllib3.
Updates
tornadofrom 6.5.8 to 6.5.9Changelog
Sourced from tornado's changelog.
... (truncated)
Commits
75ef8b1Merge pull request #3719 from bdarnell/fixes-6593590cb4test: Hardcode SimpleAsyncHTTPClient in HTTP1xxLimitTestCase9fc5d6dtest: Make tracemalloc optional in httpclient_test555a2eeiostream: Treat connection resets as a clean close in read_until_close3ba622fRelease notes and version bump for 6.5.941eea68test: Fix some test issues only found by our custom tox configab1a778Merge remote-tracking branch 'bdarnell/claude/asynchttpclient-streaming-memor...437ab5fweb: Do not follow symlinks out of the static directory0394513httputil: Apply the argument count limit to query stringsb798f83http1connection: Return after reading the response that follows a 1xxUpdates
urllib3from 2.7.0 to 2.8.0Release notes
Sourced from urllib3's releases.
... (truncated)
Changelog
Sourced from urllib3's changelog.
... (truncated)
Commits
b1d30abRelease 2.8.09016d7eSkiptest_read_chunked_with_trailing_data_does_not_hangfor brotlicffi (#5258)9101f58Fixnox -s docswarning (#5256)cd770b0Merge commit from forkea2ad7bMerge commit from fork0716e31Fix loading unencrypted client keys with a password in pyOpenSSL (#5255)43c68c8Test pickling ofInvalidChunkLength(#5247)308b279Share security policy between GitHub and Read the Docs (#5253)53fa073Add policy on duplicate pull requests (#5252)5f2a6a8Assert on the ALPN extension in test_tunnel_sets_http_11_alpn (#5232)