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.17k stars 449 forks source link

Add quirk for clarksoneyecare.com #769

Closed vcsjones closed 1 week ago

vcsjones commented 4 months ago

Overall Checklist

for password-rules.json

Collapsing unrelated checklist #### for change-password-URLs.json - [ ] There is no Well-Known URL for Changing Passwords (`https://example.com/.well-known/change-password`) - [ ] The URL either makes the experience better or no worse than being directed to just the domain in a non-logged-in state #### for shared-credentials.json - [ ] There's evidence the domains are currently related (SSL certificates, DNS entries, valid links between sites, legal documents etc.) - [ ] If using `shared`, the new group serves login pages on each of the included domains, and those login pages accept accounts from the others. (For example, we wouldn't use a `shared` association from `google.co.il` to `google.com`, because `google.co.il` redirects to `accounts.google.com` for sign in.) - [ ] If using `from` and `to`, the new group, the `from` domain(s) redirect to the `to` domain to log in. #### for shared-credentials-historical.json - [ ] You believe that the domains were associated at some point in the past and can explain that relationship

The patient portal for clarksoneyecare.com requires special characters, but of the permitted special characters, a hyphen (-) is not permitted. This results in Apple's default generated passwords not meeting the password requirements.

Here is a screenshot of the password rules:

Screenshot 2024-05-31 at 3 18 13 PM

Lowercase letters are required too, but since the website only shows you the rules on validation, I had to put something in the textbox to tell me what the requirements were, which I used a single letter "a" for that. That is why required: lower is included though the error message does not indicate it is required.

The website also (seemingly) has no maximum password length, so that was omitted.

dmmaslenn commented 1 week ago

I've updated this PR since in the password requirements for this website it is not explicitly stated that lower case characters are required, but they are allowed. @vcsjones please let us know if we missed something and lowercase characters need to be required.