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

Phyn Home Name can't be set to "Home" or you'll get an "Unknown Error" during integration setup #10

Open
opq8 opened this issue Oct 13, 2023 · 2 comments

Comments

@opq8
Copy link

opq8 commented Oct 13, 2023

First time user having an issue during the setup of the integration / Phyn:

If I enter an incorrect username/password, then I get an error about the username/password in the Logs.

If I enter the correct username/password, then I get an error about homes[0]["alias_name"]:

File "/config/custom_components/phyn/config_flow.py", line 31, in validate_input
return {"title": homes[0]["alias_name"]}
~~~~~~~~^^^^^^^^^^^^^^
KeyError: 'alias_name'

If I change
return {"title": homes[0]["alias_name"]}
to
return {"title": homes[0]}
... then the Integration adds successfully but then I only get the water consumption stats and not the toggles, and not the Phyn gets added as [object][object] or the like...

Did something change with the Phyn API?


Logger: aiohttp.server
Source: /usr/local/lib/python3.11/site-packages/aiohttp/web_protocol.py:403
First occurred: 7:22:58 PM (5 occurrences)
Last logged: 7:39:21 PM

Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
resp = await request_handler(request)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_app.py", line 504, in _handle
resp = await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/aiohttp/web_middlewares.py", line 117, in impl
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 236, in auth_middleware
return await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/headers.py", line 31, in headers_middleware
response = await handler(request)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 148, in handle
result = await handler(request, **request.match_info)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/decorators.py", line 63, in with_admin
return await func(self, request, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 177, in post
return await super().post(request, flow_id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 72, in wrapper
result = await method(view, request, data, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 293, in async_configure
result = await self._async_handle_step(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 394, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/phyn/config_flow.py", line 46, in async_step_user
info = await validate_input(self.hass, user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/phyn/config_flow.py", line 31, in validate_input
return {"title": homes[0]["alias_name"]}
~~~~~~~~^^^^^^^^^^^^^^
KeyError: 'alias_name'
@opq8
Copy link
Author

opq8 commented Oct 13, 2023

So you're all going to love this, I think there's a missing escape somewhere. My Phyn's home name (in the Phyn App > Settings > Home > Address > Home Name) was set to Home and this was causing the issue. I changed it to something else and the issue is resolved.

@opq8 opq8 changed the title API error with the homes[0]["alias_name"] during setup Phyn Home Name can't be set to "Home" or you'll get an API error with the homes[0]["alias_name"] during setup Oct 13, 2023
@opq8 opq8 changed the title Phyn Home Name can't be set to "Home" or you'll get an API error with the homes[0]["alias_name"] during setup Phyn Home Name can't be set to "Home" or you'll get an "Unknown Error" during integration setup Oct 13, 2023
@bradarnold
Copy link

I had the same issue. I changed my home name to something other than "Home", and then I could set up the integration in HA.

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