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

Config not found #8

Closed
jakob42 opened this issue Mar 23, 2017 · 3 comments
Closed

Config not found #8

jakob42 opened this issue Mar 23, 2017 · 3 comments

Comments

@jakob42
Copy link

jakob42 commented Mar 23, 2017

Hi,

I've got the problem that the pdns_hook cannot find it's config. It seems like $CONFIG is already set and thats why it doesn't load the config from the usual places (for me right next to the hook script).

I added CONFIG="" in line 26 below DIR="$(dirname "$0")", which solved it for me, but I'm not sure what other repercussions it might have and maybe somebody else has the same problem.

dehydrated.conf:

CHALLENGETYPE="dns-01"
CERTDIR="/home/dehydrated/etc/certs"
DOMAINS_TXT="${CERTDIR}/../domains.txt"
ACCOUNTDIR="${CERTDIR}/../accounts"
HOOK=/home/dehydrated/pdns_api_hook/pdns_api.sh
HOOK_CHAIN="yes"
CONTACT_EMAIL=my@email

pdns_api config

HOST=pdnsserver
PORT=8081
KEY=API_key
WAIT=60
@silkeh
Copy link
Owner

silkeh commented Mar 27, 2017

If $CONFIG is already set, what is it set to?

@jakob42
Copy link
Author

jakob42 commented Mar 27, 2017

It is set to my dehydrated.conf, adding the pdns settings there seems to work. Just not sure if it is supposed to be like that (since the config isn't prefixed and could collide with dehydrated settings). But it works for me like that, so feel free to close and sorry for the noise.

@silkeh
Copy link
Owner

silkeh commented Mar 27, 2017

The original idea was that it could either be set in the dehydrated config or a separate config file. The latter doesn't work if $CONFIG is set.

You are right in that the variables should be prefixed, I will add them for the next release.

I will keep this issue open to remind me to think about the configuration. Thanks for reporting!

@silkeh silkeh closed this as completed in 9e5ed20 May 13, 2017
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

No branches or pull requests

2 participants