Skip to content
This repository was archived by the owner on Jul 7, 2026. It is now read-only.

Fix flake8 lint failure: remove unused key_to_alg import#5527

Merged
PJ1288 merged 2 commits into
Netflix:mainfrom
PJ1288:main
Jul 6, 2026
Merged

Fix flake8 lint failure: remove unused key_to_alg import#5527
PJ1288 merged 2 commits into
Netflix:mainfrom
PJ1288:main

Conversation

@PJ1288

@PJ1288 PJ1288 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

test_acme_handler.py imported key_to_alg but never used it, tripping F401 in the lint-python CI step.

test_acme_handler.py imported key_to_alg but never used it, tripping
F401 in the lint-python CI step.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@PJ1288 PJ1288 requested a review from a team as a code owner July 6, 2026 20:14
…d error

crl_verify pins CRL requests to the DNS-resolved IP to prevent rebinding,
so the requests_mock stub must match the pinned URL rather than the
original hostname URL, or the request falls through with a
requests_mock "No mock address" error instead of the expected Timeout.

Also annotate FakePlugin.options as list to satisfy mypy's
var-annotated check.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage is 59.879%PJ1288:main into Netflix:main. No base build found for Netflix:main.

@PJ1288 PJ1288 merged commit 2b0c07b into Netflix:main Jul 6, 2026
12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants