Repository navigation
security(pip): pin black 26.3.1, urllib3 2.8.0, requests 2.34.2 in dev requirements - #100
Merged
Merged
Conversation
…v requirements requirements.txt is the README's dev-setup file. tox.ini and pyproject.toml are unchanged, so the published package and its Python >=3.8 support are unaffected. The new pins need Python >=3.10 for local dev setup. - black (unpinned, resolved by Snyk to 23.3.0) -> 26.3.1 - urllib3 2.6.3 -> 2.8.0 (2.7.0 fixes the ticketed highs; 2.8.0 also fixes SNYK-PYTHON-URLLIB3-20302844/-45/-46) - requests 2.32.4 -> 2.34.2 (SNYK-PYTHON-REQUESTS-15763443, fixed in 2.33.0) - example.py: black 26 formatting. The Lint workflow installs unpinned black, so without this, Lint fails on the next PR. Snyk (requirements.txt): 6 issues -> 0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Contributor
Author
|
Note on the red
⇒ It looks like the Platform test credentials in the |
gmurugezan
approved these changes
Oct 20, 2026
gmurugezan
approved these changes
Oct 27, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the HIGH Snyk findings in
requirements.txt(ADO/ArmorCode tickets 761395 black, 761468 urllib3).Changes
blackurllib3requestsexample.pyblack, so it already runs black 26. Without this, Lint fails on the next PR.Scope
requirements.txtis the README's dev-setup file.pyproject.toml(package metadata,requests >= 2.12.5) andtox.ini(test deps) are unchanged, so the published package and itsrequires-python >= 3.8are unaffected. Note: the new pins need Python ≥ 3.10 for local dev setup (black 26, urllib3 2.8 and requests 2.34 all require it).Results
requirements.txt, Python 3.12): 6 issues → 0 (main: 4 high + 2 medium)black --check .: passesTesting
TSS_*credentials. Locally it gives the same result onmainand this branch (Must provide either tenant or base_url). CI'sRun Testsusestox.inideps, which this PR doesn't touch.PasswordGrantAuthorizer→POST /oauth2/token,get_secret(42)→ServerSecretwith all fields and the password value, the file-attachment field fetch (/secrets/42/fields/private-key), and a missing secret raisingSecretServerClientError. Identical behaviour on both.🤖 Generated with Claude Code