Skip to content
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

digitalocean: configurable base URL #1687

Merged
merged 2 commits into from
Oct 3, 2022
Merged

Conversation

rienafairefr
Copy link
Contributor

I wanted to proxy the calls to DigitalOcean API made for dns ACME challenge, and the go-acme/lego digitalocean provider has hardcoded https://api.digitalocean.com, this PR should handle that hopefully.

@rienafairefr rienafairefr changed the title configurable base url for digitalocean digitalocean: configurable base url for provider Aug 2, 2022
@ldez ldez self-requested a review August 2, 2022 17:27
@ldez
Copy link
Member

ldez commented Aug 2, 2022

Hello,

why do you want to proxy the calls to DigitalOcean API made for DNS ACME challenge?
What kind of information did you expect?

@rienafairefr
Copy link
Contributor Author

I'm experimenting with limiting token disclosure risk by passing API calls through a passthrough proxy (similar to https://github.com/google/magic-github-proxy). This way I could put another token somewhere that would be allowed to do the necessary calls for an acme challenge on a certain domain, and not a iota more.

@rienafairefr
Copy link
Contributor Author

I could workaround using a more custom provider I think (like the httpreq), but I figured I could send the PR either way, some providers here have a customizable base_url.

@ldez ldez mentioned this pull request Aug 2, 2022
2 tasks
@ldez ldez added this to the v4.9 milestone Oct 3, 2022
@ldez ldez changed the title digitalocean: configurable base url for provider digitalocean: configurable base URL Oct 3, 2022
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ldez ldez enabled auto-merge (squash) October 3, 2022 13:20
@ldez ldez merged commit ca6784a into go-acme:master Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants