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

Not really an issue but something to add to the documentation #12

Open
sebastienjean opened this issue Nov 9, 2022 · 0 comments
Open

Comments

@sebastienjean
Copy link

Hi,

First of all, thx for this helpful integration.

I ve experienced an issue when configuring it for the first time, that is related to the way the ovh api is requested.
I finally figured out this is not a bug in the code but a side effect.

My dynhost entry had been set up using OVH manager, and i've added the ovh related data in HA configuration.yaml.
When restarting, HA was keeping complaining about a connection error raised by line 66 of __init.py__.

I finally figured out that the problem was coming from a '#' character in my dynhost username entry password, and that the presence of this character was breaking the url built with the pattern found in line 83 of the same script url = f"https://{user}:{password}@{HOST}?system=dyndns&hostname={domain}"

Maybe it should be useful to add a note in the documentation to warn about special characters usage in passwords.
My 2 cents.

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

1 participant