8000 Post 86.1 upgrade with Blink Could not retrieve videos. Message: An app update is required · Issue #20380 · home-assistant/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Post 86.1 upgrade with Blink Could not retrieve videos. Message: An app update is required #20380

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.

Sign up for GitHub

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

Jump to bottom
Closed
jlvandusen opened this issue Jan 24, 2019 · 6 comments · Fixed by #20651
Closed

Post 86.1 upgrade with Blink Could not retrieve videos. Message: An app update is required #20380

jlvandusen opened this issue Jan 24, 2019 · 6 comments · Fixed by #20651
Labels
integration: blink

Comments

@jlvandusen
Copy link
jlvandusen commented Jan 24, 2019 •
edited by cgarwood
Loading

Home Assistant release with the issue:

0.86.1

Last working Home Assistant release (if known):
0.85.1

Operating environment (Hass.io/Docker/Windows/etc.):

Raspberry Pi B+ 64bit

Component/platform:

blink integration

Description of problem:
The logs are showing every minute the following warnings:
Could not retrieve videos. Message: An app update is required
I have assigned whitelist folders etc... I don't know what has changed but Blink keeps throwing these warnings every minute.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
configuration.yaml

# Blink integration
blink:
  username: myusername
  password: !secret blink_password
#  binary_sensors:
#    monitored_conditions:
#      - motion_enabled
#      - motion_detected
#  sensors:
#    monitored_conditions:
#      - battery
#      - temperature
#      - wifi_strength

Traceback (if applicable):

Additional information:
@marchingphoenix
Copy link
Contributor

Might be related to:
PR: #20351
Issue: #20335

@dgomes
Copy link
Contributor
dgomes commented Jan 24, 2019

cc @fronzbot

@dgomes
Copy link
Contributor
dgomes commented Jan 24, 2019

I think the message should be more informative. Say that it's a current know issue.

@fronzbot
Copy link
Contributor

It's a message generated by the library since the video API is down. Casualty of the quick fix. If we can't find the new video API endpoint, I'll try to push a change to limit the message to happen only at startup by next HA release.

@fronzbot fronzbot mentioned this issue Jan 24, 2019
Closed
@fronzbot
Copy link
Contributor

Found the new endpoint, so working on implementing that fix now: fronzbot/blinkpy#143

Once that's done I plan on improving the logging (so it doesn't 💩 all over the log 😄) and implementing some back-up urls that may future proof the library for a little bit (based on some endpoint discoveries that have been made). Once all THAT is done, I'll release a new version and make the required changes to the home-assistant component. I'm confident I'll get this done by next major HA release 👍
A66B

@dgomes
Copy link
Contributor
dgomes commented Jan 25, 2019

Great news @fronzbot ! And good job!

Don't need to wait for the next release! This is breaking many setups, we will push it into a 0.86.x release.

@fronzbot fronzbot mentioned this issue Feb 1, 2019
Merged
3 tasks
@ghost ghost added the in progress label Feb 1, 2019
@ghost ghost removed the in progress label Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
integration: blink
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Upgrade blinkpy to re-enable motion detection (Fixes #20380)
5 participants
@dgomes @fronzbot @cgarwood @marchingphoenix @jlvandusen
0