-
-
Notifications
You must be signed in to change notification settings - Fork 28.8k
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
Whois integration issues #117805
Comments
Hey there @frenck, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) whois documentation |
Sadly, the current library (https://github.com/mboot-github/python-whois) is archived and probably the issues with pl domains won't get fixed. |
@mkrasowski do you have a custom integration I could try/use? EDIT: I just tested https://github.com/richardpenman/whois and it works with my domains |
The problem
The Whois integration doesn't report correct data for some TLDs. Some examples below. This list is definitely not complete.
Invalid/missing attributes for
.pl
domains (example:hl7.pl
):Invalid attributes for
.us
domains (example:test.us
):Invalid attributes for
.me
domains (example:test.me
)What version of Home Assistant Core has the issue?
core-2024.5.4
What was the last working version of Home Assistant Core?
No response
What type of installation are you running?
Home Assistant OS
Integration causing the issue
whois
Link to integration documentation on our website
https://www.home-assistant.io/integrations/whois/
Diagnostics information
config_entry-whois-001b27fcc94aeeb3cea58274b6a57b68.json
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
The python library HA is using for
whois
queries has became unsupportedhttps://pypi.org/project/whois/#description
The recommended replacement is
whoisdomain
, which has a reported memory leak - mboot-github/WhoisDomain#30, which probably disqualifies it from being used in a long-running application like Home Assistant.Using an alternative library is discussed in #117749.
The text was updated successfully, but these errors were encountered: