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

roomba.py update #37

Merged
merged 1 commit into from Aug 8, 2018
Merged

roomba.py update #37

merged 1 commit into from Aug 8, 2018

Conversation

goooroooX
Copy link
Contributor

Logging enhancement and fix for home-assistant.io.
I would also suggest to update PyPI repository.

Logging enhancement and fix for home-assistant.io.
I would also suggest to update PyPI repository.
@goooroooX
Copy link
Contributor Author

Original exception in Home Assistant (after adding more traceback):

[I 2018-07-07 16:48:50,209] Error setting TLS: Traceback (most recent call last):
  File "/opt/homeassistant/lib64/python3.6/site-packages/roomba/roomba.py", line 264, in setup_client
    tls_version=ssl.PROTOCOL_TLSv1)
  File "/opt/homeassistant/lib64/python3.6/site-packages/paho/mqtt/client.py", line 702, in tls_set
    context.load_verify_locations(ca_certs)
FileNotFoundError: [Errno 2] No such file or directory

[E 2018-07-07 16:48:50,209] Error: Must configure SSL context before using tls_insecure_set.

@goooroooX
Copy link
Contributor Author

@zacs
Copy link

zacs commented Aug 8, 2018

@NickWaterton any chance to get this merged soon? Roomba functionality is busted for Home Assistant folks at the moment. Thanks!

@NickWaterton NickWaterton merged commit 0bb2ca7 into NickWaterton:master Aug 8, 2018
@NickWaterton
Copy link
Owner

OK, Done.

@zacs
Copy link

zacs commented Aug 8, 2018

Thank you sir!

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.

None yet

3 participants