-
Notifications
You must be signed in to change notification settings - Fork 15
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
Virgin Media (GB) recording capacity #31
Comments
@callumeveratt I think this works as intended. For me there's no way to determine if cloud recording is available but to query that url. If it gives me a 501 i know it isn't avaiable. I presume you don't have a sensor at all anymore for recording capacity? |
It looks like VM sends this over MQTT as a percentage -- Topic: |
Ah m8 thats brilliant information. Until now i was only supporting cloud recordings. In this way I can add a sensor for local recordings. How did you find out? |
All the investigation I've been doing has been just by logging into https://virgintvgo.virginmedia.com with the Chrome dev tools open and watching the calls it makes. In particular, in the call to |
Cool, I'm going to try to add this to the api. Thanks! |
BTW, can you find out of anything in your account data states that recording is enabled and if this is network or local? |
Ignore that - I imagine it's not specified anywhere for Virgin GB as channel owners have cold feet over cloud recording - so it's never been enabled in the UK from what I gather |
Would definitely be cool to use the local recording data where applicable though :-) |
Will also have a dig around -- but with Virgin, I can confidently say it'll always be local as we don't have Cloud DVR! @callumeveratt I think remote recording in that context is the ability to remotely instruct your box to record something rather than recording to VM's servers |
@caraar12345 - Yeah you're right, I got ahead of myself for a second. It is the ability to remote record, not cloud record. |
Ha! Impressive timing from both of us 😁 Does look like there's something about local DVR though -- Unrelated tidbit, I do wonder where they get 'gender' from, as I may be somewhat effeminate, but I've definitely never identified myself as female to anyone 😆
|
From what call is this data coming from? |
Are you guys (or guy and lady) able to update and can you check if every box have an attribute with recording capacity? |
Will give it a shot; unfortunately I'm still contending with the config flow issue (hopefully this restart will sort it!?) |
@Sholofly On VirginMedia GB, I have 1 device and 1 entity. No sign of recording capacity anywhere. |
If you click the box entity, does it show in attributes? |
No you should check the developer tools |
Depends, do you have local recordings? |
Yep - have recordings on the box and on TVGo I can see the MQTT request @caraar12345 gave
|
ok, i that case it doesn't work yet. I have to add more logging to find out where the problem is. Do you have debugging enabled in your component? |
Nope - but if you need to add some more logging I can get it turned on in the meantime |
OK stupid. I didnt't commit the part to subscribe to the actual mqtt topic :( Can you manually update your manifest.json to
And then reboot of course ;) |
The local recordings are an attribute of the box. It's not an account thing like the cloud recordings. |
After changing the entry 0.5.12 to 0.5.13 and restarting, I no longer have an entry for recording capacity listed in states. I can see all my local recodings in the integration. |
Do you see recording capacity of each box in the box attributes? |
I updated the manifest also - the recording_capacity did disappear originally but it came back but sadly still null |
Can you check the logs for mqtt messages if you set it to debug? |
Just trawling though, so far found these
|
I see a SUBACK shortly after - but not much else |
If you visit the web page now and force the webpage to retrieve your capacity. Is your HA instance getting an update too? |
I've just re-enabled debug and restarted, SOOO much stuff in the logs I need to go through more fully, but this is a start
|
@ChrisThomas42 I believe that's the cloud recording rather than local - and that error is expected with GB as no cloud recording @Sholofly - I forced a refresh of my TV Go (if that's what you meant) and I couldn't see anything in the logs around MQTT and the localCapacity topic |
Ok just one last try for today: can you update your manifest to v0.5.14? |
Tried - didn't work originally but thought I'd relink as a test also but actually seems the integration can't connect now:
|
|
Oh crap, I’d bet you’ve maxed your session count. I had to leave it an hour
and some started clearing out so I could log in again 😅
…On Wed, 21 Dec 2022 at 15:40, callumeveratt ***@***.***> wrote:
Tried - didn't work originally but thought I'd relink as a test also but
actually seems the integration can't connect now:
Logger: custom_components.lghorizon.config_flow
Source: custom_components/lghorizon/config_flow.py:81
Integration: LG Horizon (documentation
<https://github.com/Sholofly/lghorizon>, issues
<https://github.com/Sholofly/lghorizon/issues>)
First occurred: 15:38:17 (4 occurrences)
Last logged: 15:38:47
'LGHorizonAuth' object has no attribute 'accessToken'
—
Reply to this email directly, view it on GitHub
<#31 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABOYOFGR6K6NMMUXP2VJBQDWOMQFZANCNFSM6AAAAAATFPNJ4A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Aha of course - I'll give it a little |
so this is a small extract from my log,
I think(hope) I've removed any potentially sensitive info???? There's more before and after. |
Woohoo - I finally managed to get the config flow to work so I can chime in! source_list:
- <many_channels>
media_content_type: episode
media_title: BBC Launcher
source: BBC Launcher
entity_picture_local: >-
/api/media_player_proxy/media_player.virgin_tv?token=
play_mode: app
channel: BBC Launcher
title: BBC Launcher
image: >-
https://cdn.metrological.com/lightning/apps/com.bbc.app.launcher/1.0.0-a646d9e/static/icon.png
recording_capacity: null
device_class: tv
entity_picture: >-
https://cdn.metrological.com/lightning/apps/com.bbc.app.launcher/1.0.0-a646d9e/static/icon.png
friendly_name: Virgin TV
supported_features: 154497 No MQTT message is coming through with the Interesting note: https://prod.spark.virginmedia.com/eng/web/recording-service/customers/{household_id}/recordings?sort=time&sortOrder=desc&language=en does work. And the data there contains So worst case: iterate through the recordings and add the disk spaces!??? |
Apologies as been out this evening, just got back and managed to readd. Still no luck and either no value reported for recording or a null value |
@caraar12345 We have this call too. I results in a list AND a summary of quota at the end. Isn't that the case here? |
@Sholofly - just realized we all went quiet on this. I have a bit of time this evening so I can have a look into some of the calls ect and see if I can tweak it to work on my HA if you want :) |
@callumeveratt Sorry, I'm very busy lately. Please let me know when you have time. Maybe we can figure this out soon :) |
I've started to look at this, and think my findings are the same as above. Using the Virgin web interface, I've used both Chrome developer tools and burp to capture HTTP traffic and there is no call to Looking at the JS pages (hard because of the obfuscation) there is code that looks for I can see some MQTT calls in Chrome dev tools, but the messages are only 2 bytes of binary data as far as I can see. I can't see any indication the code is iterating the recordings and adding it up. Should we (at least) put some code that says "If GB don't make the |
Ahh, just found the following MQTT message...
which contains the "used:6" - which is the correct - 6% disk used. @Sholofly if you can give me a few pointers on what this might mean, and where to modify the code, I'll give it a go. |
Sorry my time is limited nowadays so I try to respond as quickly as possible. Let me start to thank you for your dedication! I'm from NL and the Ziggo boxes don't use local recordings but cloud recordings. That's why my capacity is exposed using the API. More info: https://mqtt.org/ What you see in your example is the fact that a message is received on the /localRecordings/capacity topic. With the used percentage of your box. I think the subscription already is in place (https://github.com/Sholofly/lghorizon-python/blob/2fa57cc4ea5decff549db3fa7f952ef9403bd018/lghorizon/models.py#L363) And the processing is already in place too: So probably my cloud update is frustrating the localrecording, or there's something wrong in my processing. Probably you can figure that one out. I'd really like to hep you, but since I don't have a UK account with local recordings, that's kind of difficult for me... |
@Sholofly thank you for the quick reply, I think I can see how to make that work. The MQTT subscription to local/capacity will presumably only send a message when the capacity changes. Is there a way to force a "read" of the current status via MQTT? I appreciate your help. |
I'm not sure, probably you can inspect your browser for the answer. Since you saw the message appear, probably the previous message is the trigger? Or it's send automatically on subscription? |
Pull request created to inhibit error messages. The quota sensor seems focused on web recordings and is not available in GB. |
Pull request accepted in v0.6.3. |
If solved.... yes! |
I'm not sure this ever worked on the original Arris integration (certainly for me the value was always unavailable for recording_capacity) but the following appears in the log for getting recording capacity in GB:
The text was updated successfully, but these errors were encountered: