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

UPC Connect Box UI changed for Vodafone customers #72037

Closed
robotastic-io opened this issue May 17, 2022 · 15 comments
Closed

UPC Connect Box UI changed for Vodafone customers #72037

robotastic-io opened this issue May 17, 2022 · 15 comments

Comments

@robotastic-io
Copy link

The problem

Hi,

Vodafone updated the ui for its customers on the UPC Box so the integration is not able to login and retrieve the device information. Let me know if I can support you with any additional information.

grafik

What version of Home Assistant Core has the issue?

core-2022.5.2

What was the last working version of Home Assistant Core?

Does not matter. Target system changed

What type of installation are you running?

Home Assistant OS

Integration causing the issue

UPC Connect Box

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?

Error received on 123: Server disconnected
22:05:33 – (FEHLER) /usr/local/lib/python3.9/site-packages/connect_box/__init__.py - Die Nachricht ist zum ersten Mal um 19:14:47 aufgetreten und erscheint 342 mal
Can not login to x.x.x.x: Server disconnected
22:05:22 – (FEHLER) /usr/local/lib/python3.9/site-packages/connect_box/__init__.py - Die Nachricht ist zum ersten Mal um 19:14:47 aufgetreten und erscheint 342 mal

Additional information

No response

@probot-home-assistant
Copy link

upc_connect documentation
upc_connect source
(message by IssueLinks)

@probot-home-assistant
Copy link

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!
(message by CodeOwnersMention)

@pvizeli
Copy link
Member

pvizeli commented May 18, 2022

I guess they also change the API. The integration don't login over the UI

@fabaff
Copy link
Member

fabaff commented May 21, 2022

I don't have access to a device that uses the Vodafone modification or their firmware.

@robotastic-io
Copy link
Author

I had a look at the code and the API. The tokens are passed as a cookie now. The rest seems quite similiar apart from a few changes. I created a pull request, that should work for both the old and the new version, but I can't test it with the old version.

home-assistant-ecosystem/python-connect-box#28

I'm not a python dev so feedback is welcome

@tams9
Copy link

tams9 commented Jun 2, 2022

Hi, I can see the modified connect_box/init.py file in the link. I can see green and red highligted lines. I assume green is new code to insert and red lines are the old ones to remove.
I was looking for connect_box/init.py file in my Home Assistant server to update manually and try the modifications, but did not find the same named file.

Could you please help me applying the changes?

Unfortunately in Hungary also Vodafone upgraded the firmware, which destroyed my presence detection.

Thank you in advance!

@github-actions github-actions bot added stale and removed stale labels Jul 15, 2022
@philharmonie
Copy link

Any updates?

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@fabaff
Copy link
Member

fabaff commented Mar 4, 2023

Without access to a device that uses the firmware in question there is not much I do. Usually the providers do not publish documentation or know what they have as they do not do the firmware development by themself.

Pretty much the only way get support is if one contributes to the Python module or me having permanent access to a device.

@github-actions github-actions bot removed the stale label Mar 4, 2023
@kvalev
Copy link

kvalev commented Mar 4, 2023

@fabaff Already did home-assistant-ecosystem/python-connect-box#31 ;)

I also have the patch for the upc_connect plugin ready, but first the above PR needs to be merged and a new version of the library needs to be released to PyPI.

For what is worth I am running my patched version for about a week now without any problems.

@birdie60
Copy link

I have the same errors "Error received on 123" and "Error received on 123: Server disconnected" from components/upc_connect/device_tracker.py:69 in my log many times and since long.
Pls tell me if I can be of help!

@kvalev
Copy link

kvalev commented Mar 27, 2023

@birdie60 Until the fix is upstreamed you can use my patched integration. Details how to install it are in the readme. https://github.com/kvalev/upc_connect

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@birdie60
Copy link

@birdie60 Until the fix is upstreamed you can use my patched integration. Details how to install it are in the readme. https://github.com/kvalev/upc_connect

Thanks, kvalev! FYI, I installed upc_connect v2, but I still get this in my logs:
Error received on 123:

@issue-triage-workflows
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@issue-triage-workflows issue-triage-workflows bot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 1, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Oct 31, 2023
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

7 participants