Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Cannot retrieve videos

See original GitHub issue

Describe the bug Blink’s endpoint for video retrieval changed, yet again. Any access to the currently implement endpoint results in old videos being returned (nothing new since roughly May 13th).

blinkpy version: 0.13.1

Log Output/Additional Information New api endpoint for api.request_videos method: /api/v1/accounts/<account_id>/media/changed?since=<time>&page=<page>

In the blinkpy._parse_downloaded_items method, the following keys have changed:

camera_name --> device_name
address --> media

Those should be the only required changes to get this method working again. The test_blink_functions.py test will also need to be updated with the new keys (around lines 122-127 and 129)

Issue Analytics

State:
Created 4 years ago
Comments:11 (5 by maintainers)

Top GitHub Comments

1reaction

sutekh137commented, May 20, 2019

Yeah, it definitely works for getting recent videos. Though, blinkpy doesn’t work on Windows with regard to using the download_videos() method. The file name constructed doesn’t work in Windows. At some point it might be nice to use an OS-specific tweak of the file name to make it valid for whatever OS it is running on. Even with that, I could see the timestamp of the video found and it was recent (and getting more and more recent on subsequent runs).

Thanks much!

0reactions

sutekh137commented, May 20, 2019

A bug, or a feature? smile I am sure Microsoft would call it the latter, though I suppose the Linux file system has its quirks, too.

Thanks, sutekh137

On Mon, May 20, 2019 at 9:34 AM Kevin Fronczak notifications@github.com wrote:

Thanks for checking and letting me know about the Windows bug! I think I figured out the problem and will push out a fix today

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/fronzbot/blinkpy/issues/170?email_source=notifications&email_token=AA6INAKKKIMJKICYEEJJOTTPWKZGHA5CNFSM4HNNW7WKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVZAXTQ#issuecomment-494013390, or mute the thread https://github.com/notifications/unsubscribe-auth/AA6INAMKHTW3JI2JDYKZVVLPWKZGHANCNFSM4HNNW7WA .