Skip to content

Add safeway.com password quirks #899

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

Merged
merged 1 commit into from
May 5, 2025

Conversation

taylortrimble
Copy link
Contributor

From the password reset page:

  • Between 8–40 characters.
  • At least one upper case letter.
  • At least one symbol (!@#$%^&*).

Tested with all provided symbols, including at the front of the password, and with passwords not containing any digits or lower case letters.

Screenshot:
Screenshot 2025-05-02 at 11 38 57 PM

Overall Checklist

for password-rules.json

  • The given rule isn't particularly standard and obvious for password managers
  • Generated passwords have been tested from this rule using the Password Rules Validation Tool
  • Information has been included about the website's requirements (eg. screenshots, error messages, steps during experimentation, etc.)
  • The PR isn't documenting something that would be a common practice among password managers (e.g. minimal length of 6)

@taylortrimble
Copy link
Contributor Author

Note: because of safeway.com rules, default passwords generated by Safari like:

  • tyckyp-4bUhxe-jymroz
  • nahgeN-4huwdi-gadjos
  • wumky6-hippef-kubHux

are not allowed because hyphens aren't allowed.

@taylortrimble taylortrimble force-pushed the safeway-password-rules branch 2 times, most recently from 9ed7635 to 21b9aa1 Compare May 3, 2025 09:18
From the password reset page:
- Between 8–40 characters.
- At least one upper case letter.
- At least one symbol (!@#$%^&*).

Tested with all provided symbols, including at the front of the password, and with passwords not containing any digits or lower case letters.
@taylortrimble taylortrimble force-pushed the safeway-password-rules branch from 21b9aa1 to ea8bf1b Compare May 3, 2025 09:59
@rmondello rmondello merged commit e628b84 into apple:main May 5, 2025
5 checks passed
@rmondello
Copy link
Contributor

We should consider mirroring these rules to all of the other Albertsons websites.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants