Skip to content

Support multiple CloudFlare accounts#42

Open
pjeby wants to merge 2 commits into
kappataumu:masterfrom
pjeby:multiauth
Open

Support multiple CloudFlare accounts#42
pjeby wants to merge 2 commits into
kappataumu:masterfrom
pjeby:multiauth

Conversation

@pjeby

@pjeby pjeby commented Aug 15, 2017

Copy link
Copy Markdown

This PR implements support for multiple CloudFlare accounts, by trying credentials in order until an account is found that matches the current domain. Documentation is included in the patch, but basically you can add multiple, space-separated emails and api keys in the environment variables. The change is backward-compatible with existing environment variables. I tried to keep the code changes to the absolute minimum; hope you find it helpful.

@YasharF

YasharF commented May 18, 2019

Copy link
Copy Markdown

Since this repo is no longer maintained by its owner, now there is a new maintained fork at https://github.com/SeattleDevs/letsencrypt-cloudflare-hook. I tried cherry-picking your PR, and manual test to see if it is a breaking change was fine. However, it looks like the unit tests need to be modified in the other repo for this since the tox build test failed. Would you mind opening your PR against that repo and addressing the build issue as part of your PR? I would be happy to merge the change shortly afterward.

@pjeby

pjeby commented May 18, 2019

Copy link
Copy Markdown
Author

It looks like the issue is that this PR changes the call signature of _get_txt_record_id() and return signature of _get_zone_id, but the new unit tests in that repo are dependent on the old signatures (because apparently they're testing private functions?).

@YasharF

YasharF commented May 18, 2019

Copy link
Copy Markdown

I took the unit tests that were in another PR since there were no tests, to begin with, without going through them much since they passed out of the box. I would be happy to accept PRs to modify the tests.

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