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

Settings of type "password" triggers the browser password store #6408

Closed
JohnXLivingston opened this issue May 21, 2024 · 4 comments
Closed
Labels
Status: Waiting for answer Waiting issue author answer

Comments

@JohnXLivingston
Copy link
Contributor

JohnXLivingston commented May 21, 2024

Describe the current behavior

When a plugin has settings of type "password", and you save the plugin settings, the browser password store is triggered:

image

Steps to reproduce

  1. Install a plugin with such settings (livechat for example)
  2. Enter a password
  3. Save the form

Describe the expected behavior

The <input type="password"> should have autocomplete="off" to prevent this behaviour.

Additional information

  • PeerTube instance:
    • Version: 6.1.0 (and previous)
@Chocobozzz
Copy link
Owner

Hi,

We should already use autocomplete="off". Can you confirm?

@Chocobozzz Chocobozzz added the Status: Waiting for answer Waiting issue author answer label May 29, 2024
@JohnXLivingston
Copy link
Contributor Author

Hi,

We should already use autocomplete="off". Can you confirm?

Oh, right. The attribute is set.
So, why does Firefox want to save the password? Any idea?

Can you reproduce?
You can try with the livechat plugin, by enabling «open id connect», and using following testing options:

  • url: https://oidctest.wsweet.org/.well-known/openid-configuration
  • client id: public
  • client secret: tardis

@Chocobozzz
Copy link
Owner

So, why does Firefox want to save the password? Any idea?

See https://www.stefanjudis.com/snippets/turn-off-password-managers/

I think we should not try to find a workaround to prevent this behaviour web browser explicitly want

@JohnXLivingston
Copy link
Contributor Author

See https://www.stefanjudis.com/snippets/turn-off-password-managers/

Oh...

I think we should not try to find a workaround to prevent this behaviour web browser explicitly want

I found this behaviour really annoying, as it keeps asking everytime i want to save settings.
But ok, i understand that this is a browser issue.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Waiting for answer Waiting issue author answer
Projects
None yet
Development

No branches or pull requests

2 participants