new: use the public suffix list instead of tlds only (*BREAKING*) #4

Closed
heiko wants to merge 0 commits from dev into master
Owner

This compares the domain with a longer list, the list of public suffixes. It is IMHO more reliable than just checking the TLD. Though I didn't find a domain, where the TLD itself isn't public.

This compares the domain with a longer list, the list of public suffixes. It is IMHO more reliable than just checking the TLD. Though I didn't find a domain, where the TLD itself isn't public.
I did some refactoring of the existing code, hopefully
in a sensible way.

Whether we'll use GeneratePassword() or GeneratePassword2()
I don't know. IMHO I like the latter, not because it's mine, but because
it is deterministic.
* origin/dev+genpw: (40 commits)
  src: add missing licenses
  testing: test the timeout feature of GeneratePassword
  refactor, add GeneratePassword2 with deterministic behaviour
  fix: GeneratePassword start with empty password string on each iteration
  chg: use a deadline instead of a count
  chg: do not panic
  new: alternative password generator
  git: rm tlds.go
  chg: rework password check
  testing: add umlaut tld, and invalid TLD
  new: use the public suffix list instead of tlds only (*BREAKING*)
  src: style
  lint
  fmt: fileformat
  chg: remove redundancy from pwnd return values
  src: readability (maybe)
  testing: speedup
  Generator test (fixes #3)
  pwned Test hinzugefügt (fixes #3)
  src: return value name
  ...
Author
Owner

already merged

already merged
heiko closed this pull request 2025-06-10 21:17:04 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
IUS/validator!4
No description provided.