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

Added rule for account.idm.telekom.com #838

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hatobi
Copy link
Contributor

@hatobi hatobi commented Oct 24, 2024

Added a rule for password-change site of t-online.de as well as telekom.de – they both use the same subdomain for password-changes as well as the same password rules.

Rules have been tested using passwords generated by the password rules validation tool.

2024-10-24_123018_Screenshot

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)

@rmondello
Copy link
Contributor

Were you able to test that all special characters are supported? Usually websites do not support all special characters, even if they say they do.

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