Add and Update several password change urls

apple / password-manager-resources

A place for creators and users of password managers to collaborate on resources to make password management better.

MIT License

4.23k stars 455 forks source link

Add and Update several password change urls #665

Closed joshka closed 1 year ago

joshka commented 1 year ago

Overall Checklist

[x] I agree to the project's Developer Certificate of Origin
[x] The top-level JSON objects are sorted alphabetically
[x] There are no open pull requests for the same update

for change-password-URLs.json

[x] There is no Well-Known URL for Changing Passwords (https://example.com/.well-known/change-password)
[x] The URL either makes the experience better or no worse than being directed to just the domain in a non-logged-in state

simap-ag commented 1 year ago

Can you remove commit 03e1675, as I have it in another pull request already.

joshka commented 1 year ago

Can you remove commit 03e1675, as I have it in another pull request already.

Done.

joshka commented 1 year ago

Removing www seems generally wrong to me on these. The sites in question have chosen to use the prefix as their canonical url (all urls are copied from a live session using the site). The non-www version might work in some cases, but is not the url that the sites advertise themselves. I get the arguments both ways on no-www and yes-www sides of the issue (I lean no-www, but not enough to try to argue for it), but the decision is the target website owner’s not the browser / tool maker’s.

This said - it’s entirely possible there’s something technical I’m missing that’s not stated in why you want the www dropped - help me understand if I have?

rmondello commented 1 year ago

@joshka If these are, in fact, the canonical URLs, then this is great!

rmondello commented 1 year ago

Thanks for the contribution, @joshka!