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

WARNING (Recorder) Event is not JSON serializable: <Event platform_discovered[L]: #336

Closed
alandtse opened this issue Sep 3, 2019 · 4 comments · Fixed by #348
Closed

Comments

@alandtse
Copy link
Owner

alandtse commented Sep 3, 2019

Describe the bug
Recorder provides a json warning because of passing of discovered info.

To Reproduce
Steps to reproduce the behavior:

  1. Enable recorder for platform_loaded events
  2. Update to 2.1.0
  3. See error

Expected behavior
No warning

Screenshots
If applicable, add screenshots to help explain your problem.

System details

  • Home-assistant (version): 0.98.2
  • Hassio (Yes/No): No
  • alexa_media (version from const.py or HA startup): 2.1.0
  • alexapy (version from pip show alexapy or HA startup): 1.0.2

Additional context
This only appears at startup for all four components.

  • media_player
  • switch
  • notify
  • alarm_control_panel
@sephrioth
Copy link

Same here.

@lockyt
Copy link

lockyt commented Sep 3, 2019

ditto

(I had also lost my install from HACS for some reason, although alexa was still working. Just had to hit install again but was a bit weird - hence why now on 2.1.0)

@ehendrix23
Copy link

FYI, same here. :-)

@minglecm
Copy link

minglecm commented Sep 4, 2019

Seeing this as well.

It's a warning so setting the recorder component logger to > warning should silence it enough to not have your password all out there in plaintext:

logger:
  logs:
    homeassistant.components.recorder: error

alandtse added a commit that referenced this issue Sep 7, 2019
fix: clean discovery_info of unneeded values prior to passing
This was referenced Sep 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants