home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
71.54k stars 29.91k forks source link

Ziggo / UPC: Invalid start byte #105249

Closed DevSecNinja closed 5 months ago

DevSecNinja commented 9 months ago

The problem

The Ziggo / UPC Connect integration throws the following error. Could be Ziggo changed something in their Connect Box?

Logger: homeassistant.components.device_tracker
Source: components/device_tracker/legacy.py:294
Integration: Apparaattracker (documentation, issues)
First occurred: 19:11:53 (1 occurrences)
Last logged: 19:11:53

Error setting up platform legacy upc_connect
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/device_tracker/legacy.py", line 294, in async_setup_legacy
    scanner = await self.platform.async_get_scanner(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/upc_connect/device_tracker.py", line 43, in async_get_scanner
    await connect_box.async_initialize_token()
  File "/usr/local/lib/python3.11/site-packages/connect_box/__init__.py", line 380, in async_initialize_token
    await response.text()
  File "/usr/local/lib/python3.11/site-packages/aiohttp/client_reqrep.py", line 1147, in text
    return self._body.decode(  # type: ignore[no-any-return,union-attr]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa3 in position 7389: invalid start byte

What version of Home Assistant Core has the issue?

core-2023.12.0

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

upc_connect

Link to integration documentation on our website

https://www.home-assistant.io/integrations/upc_connect/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 9 months ago

Hey there @pvizeli, @fabaff, mind taking a look at this issue as it has been labeled with an integration (upc_connect) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `upc_connect` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign upc_connect` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


upc_connect documentation upc_connect source (message by IssueLinks)

Mbussie commented 9 months ago

The same here...

luka6000 commented 9 months ago

same here with Connect Box from UPC/Play in Poland. Last working version was 2023.11.3

2023-12-15 10:21:31.787 ERROR (MainThread) [homeassistant.components.device_tracker] Error setting up platform legacy upc_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/device_tracker/legacy.py", line 294, in async_setup_legacy
scanner = await self.platform.async_get_scanner(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/upc_connect/device_tracker.py", line 43, in async_get_scanner
await connect_box.async_initialize_token()
File "/usr/local/lib/python3.11/site-packages/connect_box/__init__.py", line 380, in async_initialize_token
await response.text()
File "/usr/local/lib/python3.11/site-packages/aiohttp/client_reqrep.py", line 1147, in text
return self._body.decode( # type: ignore[no-any-return,union-attr]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa3 in position 7389: invalid start byte
HeliAmmann commented 9 months ago

same

krzysztof-kwitt commented 8 months ago

It is covered by https://github.com/home-assistant-ecosystem/python-connect-box/pull/32 Can you check if this fix works for you?

DevSecNinja commented 8 months ago

It is covered by home-assistant-ecosystem/python-connect-box#32 Can you check if this fix works for you?

Thanks, that could be it! Appreciate you've looked into it! I'm not using the integration myself and I can't easily access that instance. Therefore, my opening post was also quite limited. Hope someone else can test it. I can definitely test it if it gets released because I can ask to update HA to the latest version.

Mbussie commented 8 months ago

It is covered by home-assistant-ecosystem/python-connect-box#32 Can you check if this fix works for you?

I would like to test this, but can you explain how do I implement / change this, or is it in a feature release in HA?

maikischa commented 7 months ago

Same problem here in Austria with Magenta

fabaff commented 7 months ago

I can say with the issue for Austria will be addressed with 0.3.1 as I don't have access to such a device.

mawek commented 7 months ago

I'm getting the same error (Slovakia UPC)

Core: 2024.1.6 Supervisor: 2023.12.1 Operating System: 11.5