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

Non-fatal errors on CentOS6 #114

Closed
011121 opened this issue Dec 14, 2016 · 4 comments
Closed

Non-fatal errors on CentOS6 #114

011121 opened this issue Dec 14, 2016 · 4 comments

Comments

@011121
Copy link

011121 commented Dec 14, 2016

On CentOS6 I'm seeing some nonfatal errors pop up when using tldextract:

python3.5/site-packages/tldextract/tldextract.py:223: ResourceWarning: unclosed <ssl.SSLSocket fd=5, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('10.32.216.57', 59070), raddr=('63.245.213.24', 443)>

  raw_suffix_list_data = find_first_response(self.suffix_list_urls)

python3.5/site-packages/tldextract/tldextract.py:295: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead

  LOG.warn("unable to cache TLDs in file %s: %s", self.cache_file, ioe)

unable to cache TLDs in file python3.5/site-packages/tldextract/.tld_set: [Errno 13] Permission denied: 'python3.5/site-packages/tldextract/.tld_set'

I've redacted some information from the file paths above but nothing important, I believe.

@john-kurkowski
Copy link
Owner

I'm curious about the ResourceWarning. I can't tell what it is at a glance. Would sure like to squash it.

The DeprecationWarning is a no brainer how to fix. Good catch.

The Permission denied is indeed expected if you do not have write perms to that folder. See also this comment. Maybe you installed tldextract with sudo? Can you make that folder writable by the same user you're tldextract running as? Or don't install it with sudo etc.

@john-kurkowski
Copy link
Owner

See also the caching section of the readme, to specify an alternate path to cache in.

@011121
Copy link
Author

011121 commented Dec 17, 2016

Yeah, I wasn't too worried about the permission error, I just included it for completeness.

@john-kurkowski
Copy link
Owner

Deprecation was fixed in #104. Unclosed python-requests session fixed in #140.

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