-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support alternative Cloudflare API token file format #714
Comments
@naanlizard Thank you. I will have to think about this due to the additional complexity, and this feature might have lower priority on my list (sorry). On the other hand, I wonder if your true concern is the burden to maintain consistency across multiple configuration files, which I deeply sympathize! If so, I think tools like conf might help. |
@naanlizard I wonder if you have tried the tool I suggested? In the meanwhile, I might have come up with a good solution. I wonder if you are okay with writing a regular expression to identify the token? You will need to write a regular expression and the first submatch named
Design question: the first config name that comes to my mind is ridiculously long: |
@naanlizard Hi, I have been trying various experiments and wonder how you feel about the design. BTW, I think the idea "first submatched named
and it should probably be |
I'm sorry to be so quiet here - for the moment I can't play with this, but in a few weeks I'll be redoing one of my sites and can go in depth then! |
@naanlizard I wonder if you have any new discovery? |
I wasn't quite sure how to use those regex you provided, when I was
reconfiguring things just now.
…On Fri, Sep 13, 2024 at 5:22 AM favonia ***@***.***> wrote:
@naanlizard <https://github.com/naanlizard> I wonder if you have any new
discovery?
—
Reply to this email directly, view it on GitHub
<#714 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABV4BWMNRFBYNRETJWRMB53ZWJR77AVCNFSM6AAAAABE2I2ZLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBYGAYDKMJQG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@naanlizard Sorry but the feature was not ready yet. I was mostly only brainstorming. |
@naanlizard I should elaborate: I thought you were planning to try out tools like https://github.com/kelseyhightower/confd that can synchronize multiple configuration files. Given my commitment to backward compatibility, I'd like to ensure we have a good design before implementing anything in the updater. Sorry for my miscommunication that made you think the feature was ready. I was hoping that you could join the conversation to help me pin down a good design. |
Small issue here, other services (letsencrypt and stuff using it) tend to use cloudflare's API key in the form of
https://certbot-dns-cloudflare.readthedocs.io/en/stable/
would it be possible to add support for that formatting so I don't need as many files kicking around?
Originally posted by @naanlizard in #668 (comment)
The text was updated successfully, but these errors were encountered: