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

Add predefined sites option #1010

Merged
merged 1 commit into from
Nov 1, 2020
Merged

Add predefined sites option #1010

merged 1 commit into from
Nov 1, 2020

Conversation

varjolintu
Copy link
Member

@varjolintu varjolintu commented Sep 13, 2020

Adds option for predefined sites which are added to Site Preferences at the extension start. This feature is enabled by default to ensure a better compatibility with most common sites. Duplicate URL's are not appended to the list.

Another tweaks:

  • Move generic scripts from the root directory to common/
  • Remove kpxc.initSitePreferences() and do the same check at the extension start

Please report any common sites you know that are missing from the list.

Currently the list is:

  • accounts.google.com
  • www.paypal.com
  • *.live.com
  • amazon.com
  • amazon.de
  • amazon.co.uk
  • signin.aws.amazon.com
  • www.upwork.com/ab/
  • home.personalcapital.com
  • auth.services.adobe.com
  • idmsa.apple.com
  • soundcloud.com
  • icloud.com
  • signin.ebay.de
  • signin.ebay.com
  • login.yahoo.com
  • (All other eBay domains)
  • *.atlassian.com

Fixes #1033.
Fixes #1042.
Fixes #1073.

keepassxc-browser/_locales/en/messages.json Outdated Show resolved Hide resolved
keepassxc-browser/common/sites.js Outdated Show resolved Hide resolved
@Ouack23
Copy link

Ouack23 commented Sep 27, 2020

https://soundcloud.com/ should be added with the "Detect username only" option IMO.
I also have a similar rule for https://www.oui.sncf/

@droidmonkey
Copy link
Member

Can we include an easy way for people to suggest sites to be added to the predefined list? Perhaps lead them to GitHub and a pre-filled issue template?

@varjolintu
Copy link
Member Author

@droidmonkey That would be nice.

@varjolintu
Copy link
Member Author

I need to make some more Google-related fix to this. Hold on..

@Ouack23
Copy link

Ouack23 commented Oct 21, 2020

We need a rule for https://login.yahoo.com/*
Thanks for all your work !

@nbartos
Copy link

nbartos commented Oct 28, 2020

Please also add signin.ebay.com.au

keepassxc-browser/background/page.js Outdated Show resolved Hide resolved
keepassxc-browser/background/page.js Show resolved Hide resolved
keepassxc-browser/common/sites.js Show resolved Hide resolved
@varjolintu varjolintu merged commit 82e32d4 into develop Nov 1, 2020
@varjolintu varjolintu deleted the feature/predefined_sites branch November 1, 2020 18:20
@nbartos nbartos mentioned this pull request Nov 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants