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

Chrome incorrectly auto-completes "include_file" using email address of listmaster #1033

Closed
dpc22 opened this issue Nov 16, 2020 · 5 comments · Fixed by #1563
Closed

Chrome incorrectly auto-completes "include_file" using email address of listmaster #1033

dpc22 opened this issue Nov 16, 2020 · 5 comments · Fixed by #1563
Labels

Comments

@dpc22
Copy link
Contributor

dpc22 commented Nov 16, 2020

Version

6.2.58

Installation method

RHEL 7 RPM based on:

https://dl.fedoraproject.org/pub/epel/7/SRPMS/Packages/s/sympa-6.2.58-1.el7.src.rpm

Expected behaviour

Should be able to click "Apply modifications" button at bottom of

Edit List Congigurations
-> Data sources setup

without automatically adding an "include_file" which corresponds to email address of listmaster.

Unsurprisingly that doesn't work, and an error is reported whenever you try to synchronise the data source.

Actual behaviour

include_file set to email address used to log in as listmaster unless I manually clear the value of "include_file" before clicking "Apply modifications".

Additional information

The first time that I log into Sympa as the listmaster, most Web browsers (at least Chrome/Firefox/Safari) offer to save the username and password combination.

If I then navigate to:

Edit List Congigurations
-> Data sources setup

the saved username and password values auto-completes various fields.

In Chrome:

File inclusion
(include_file)(default)
Remote file inclusion (include_remote_file)(default)
remote password (passwd)

In Firefox and Safari:

Remote file inclusion (include_remote_file)(default)
remote user (user)
 remote password (passwd)

Which is still wrong, but at least has no effect when you click "Apply modifications", unless some information is provided for the other two attributes in the same section, namely:

short name for this source (name)
 data location URL (url)

It isn't clear to me whether the problem is with Chrome or Sympa, but it would be useful if we could stop Web browsers from trying to auto-fill these fields in the first place. It looks like we just need to set a HTML attribute "autocomplete=off" :

https://developer.mozilla.org/en-US/docs/Web/Security/Securing_your_site/Turning_off_form_autocompletion

@ikedas
Copy link
Member

ikedas commented Nov 17, 2020

@ikedas ikedas added the bug label Nov 17, 2020
@dpc22
Copy link
Contributor Author

dpc22 commented Nov 17, 2020

Thanks.

We are probably going to have quite a lot of listmasters (5 to 10?) compared to most Sympa installers so that our service desk front line staff can create/approve new mailing lists.

While I can certainly warn them about the issue, there is some potential for confusion. It certainly confused me until I realised what was going on.

@ikedas

This comment was marked as outdated.

@ikedas
Copy link
Member

ikedas commented Dec 29, 2022

@dpc22, with the PR above, Chrome seems not filling include_file field automatically (I confirmed with Chrome 108). However, the e-mail address selection pull-down still appears.
image

If this is satisfactory, I'd like to close this issue.

@dpc22
Copy link
Contributor Author

dpc22 commented Jan 3, 2023

Thank you. My Javascript is quite limited, but this certainly looks plausible.

@ikedas ikedas added the ready A PR is waiting to be merged. Close to be solved label Jan 4, 2023
@ikedas ikedas removed the ready A PR is waiting to be merged. Close to be solved label Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants