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

Alarm.com broken in 0.42.2 #7049

Closed
jordemort opened this issue Apr 11, 2017 · 4 comments
Closed

Alarm.com broken in 0.42.2 #7049

jordemort opened this issue Apr 11, 2017 · 4 comments

Comments

@jordemort
Copy link

Home Assistant release (hass --version):

Docker container homeassistant/home-assistant:0.42.2

Python release (python3 --version):

Python 3.5.3

Component/platform:

alarm_control_panel.alarmdotcom

Description of problem:

Alarm.com component fails during setup.

Expected:

Alarm.com component doesn't fail during setup.

Problem-relevant configuration.yaml entries and steps to reproduce:

alarm_control_panel panel:
  platform: alarmdotcom
  username: xxxx
  password: xxxx
  1. Set up panel in configuration.yaml
  2. Start home-assistant

Traceback (if applicable):

17-04-11 10:17:54 ERROR (MainThread) [homeassistant.components.alarm_control_panel] Error while setting up platform alarmdotcom
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/site-packages/pyalarmdotcom/pyalarmdotcom.py", line 120, in async_login
    response.url).groupdict()['sessionKey'],
TypeError: expected string or bytes-like object
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/src/app/homeassistant/helpers/entity_component.py", line 148, in _async_setup_platform
    entity_platform.async_schedule_add_entities, discovery_info
  File "/usr/src/app/homeassistant/components/alarm_control_panel/alarmdotcom.py", line 43, in async_setup_platform
    yield from alarmdotcom.async_login()
  File "/usr/src/app/homeassistant/components/alarm_control_panel/alarmdotcom.py", line 69, in async_login
    yield from self._alarm.async_login()
  File "/usr/local/lib/python3.5/site-packages/pyalarmdotcom/pyalarmdotcom.py", line 132, in async_login
    except (asyncio.TimeoutError, aiohttp.errors.ClientError):
AttributeError: module 'aiohttp' has no attribute 'errors'

Additional info:

This worked fine with the 0.41 container, and apparently still does since someone else said they'd rolled back and it fixed it. I haven't rolled back because I want the shiny new Ring stuff. Forum discussion here: https://community.home-assistant.io/t/alarm-com-broken-since-upgrade-to-0-42/15346

@pvizeli
Copy link
Member

pvizeli commented Apr 11, 2017

Please open a issue on pyalarmdotcom and ask for aiohttp2 support. We can not fix that.

@jordemort
Copy link
Author

Looks like it is fixed in pyalarmdotcom 0.3.0: lorddaren/pyalarmdotcom@763fc77

@jordemort
Copy link
Author

Yeah, I was able to hack a fix for it by upgrading pyalarmdotcom in the container (something like docker exec -t -i {your container} pip install --target /config/deps --upgrade pyalarmdotcom and then restart your container, if anyone else wants to hack theirs until it's properly fixed)

@jordemort
Copy link
Author

Ah, this is the actual fix for it, guess I should have looked at the PRs as well as the issues! #7021 (thanks @Xorso :D)

@home-assistant home-assistant locked and limited conversation to collaborators Jul 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants