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

Password is shown in log #519

Closed
yjlin0224 opened this issue Feb 18, 2024 · 3 comments
Closed

Password is shown in log #519

yjlin0224 opened this issue Feb 18, 2024 · 3 comments

Comments

@yjlin0224
Copy link

yjlin0224 commented Feb 18, 2024

Description

(Edit: image is removed)

Camera is using Tapo C125.

Reproduction Steps

After I cannot see the stream of camera in Home Assistant, I open the log to see what happened, I noticed camera reported 400 error and then I saw my password in my log.

Expected behavior

All passwords should be hiden in log.

If applicable, add error logs.

No response

Device Firmware

1.1.7 Build 231222 Rel. 64156n

Integration Version

5.4.14

Using stream component

Yes

Does camera work via official integrations?

Yes

Camera has all attributes filled out in developer tools

Yes

HASS Environment

Docker with WSL2 backend in Windows 11

Search for similar issues

Yes

Additional information

No response

@JurajNyiri
Copy link
Owner

JurajNyiri commented Feb 18, 2024

RTSP requires password, in order to pass the stream to HA it needs to pass password as well.

These logs are not coming from this integration. I recommend opening ticket to HA.

@JurajNyiri
Copy link
Owner

Also the blur you used can be reversed. I recommend changing password and using solid color next time.

@yjlin0224
Copy link
Author

yjlin0224 commented Feb 18, 2024

OK, I will edit the post to remove the image and reopen the issue to HA.
After I found this issue, I immediately changed the password with password generator.

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

No branches or pull requests

2 participants