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

README: Add example 'ip' in example API config. #44

Merged
merged 1 commit into from
Feb 28, 2018

Conversation

cpu
Copy link
Contributor

@cpu cpu commented Feb 28, 2018

acme-dns supports binding the API to a specific interface instead of
all interfaces by providing an ip address in the [api] configuration
section. Prior to this commit the ip field wasn't shown in the example
configuration in the README. This commit adds an example showing how to
configure acme-dns to listen only on 127.0.0.1 to the example config
and describes what the default value ("") does.

`acme-dns` supports binding the API to a specific interface instead of
all interfaces by providing an `ip` address in the `[api]` configuration
section. Prior to this commit the `ip` field wasn't shown in the example
configuration in the README. This commit adds an example showing how to
configure `acme-dns` to listen only on `127.0.0.1` to the example config
and describes what the default value (`""`) does.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.373% when pulling 77b5fda on cpu:cpu-add-api-ip-example-config into 830cceb on joohoi:master.

@joohoi
Copy link
Owner

joohoi commented Feb 28, 2018

Thanks for the PR, and good catch!

@joohoi joohoi merged commit 32608e9 into joohoi:master Feb 28, 2018
@cpu cpu deleted the cpu-add-api-ip-example-config branch February 28, 2018 19:27
jacobmyers-codeninja pushed a commit to jacobmyers-codeninja/acme-dns that referenced this pull request Sep 30, 2020
README: Add example 'ip' in example API config.
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.

3 participants