Skip to content

update python deps and fix cloudflare module name (fix #5313)#5314

Open
gijzelaerr wants to merge 5 commits intoNetflix:mainfrom
gijzelaerr:update-deps-and-fix-cloudflare
Open

update python deps and fix cloudflare module name (fix #5313)#5314
gijzelaerr wants to merge 5 commits intoNetflix:mainfrom
gijzelaerr:update-deps-and-fix-cloudflare

Conversation

@gijzelaerr
Copy link
Copy Markdown
Contributor

This updates the python requirement files and fixes a changes module name for the cloudflare package.

@gijzelaerr gijzelaerr changed the title update python deps and fix cloudflare module name update python deps and fix cloudflare module name (fix #5313) Sep 16, 2025
Fix RST footnote reference warning in production documentation

 - Change auto-footnote [*] to numbered footnote [1] in docs/production/index.rst:495
 - Resolves Sphinx warning: "Footnote [*] is not referenced"
 - Maintains same content and functionality with proper reStructuredText syntax
…ableX509 mock

- Remove outdated mock for josepy.util.ComparableX509 which no longer exists in current josepy version
- Update test to properly mock OpenSSL.crypto functions that are actually used in the implementation
- Mock acme_client methods (poll_authorizations, finalize_order) to match real usage
- Test now verifies correct return tuple structure instead of unused mocks
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 60.861%. remained the same
when pulling 324a8ce on gijzelaerr:update-deps-and-fix-cloudflare
into 8ecc5c5 on Netflix:main.

@gijzelaerr
Copy link
Copy Markdown
Contributor Author

Hi — just checking in on this PR. It's been open for a while and has picked up merge conflicts in the requirements files. I'm happy to rebase it on the latest main and resolve the conflicts if you're interested in merging it.

The core change is small: the CloudFlare package was renamed to cloudflare (lowercase) in newer versions, which breaks the ACME DNS plugin (see #5313). The requirements updates were needed at the time to make CI pass but could be split into a separate PR if you'd prefer to keep this focused on just the module rename fix.

Let me know how you'd like to proceed — happy to rework this however works best.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants