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

Update docs to point at new fork of zxcvbn #177

Closed
wants to merge 1 commit into from

Conversation

xcskier56
Copy link
Contributor

I found that there is a new fork of the zxcvbn library for the frontend. This has some nice features like optionally not including the dictionaries to reduce file size.

@xcskier56 xcskier56 changed the title Update to point at new fork of zxcvbn Update docs to point at new fork of zxcvbn Jun 9, 2021
@cainlevy
Copy link
Member

Nice, I sure do appreciate modernized and supported frontend libraries. 😂

The dilemma here is that the backend's zxcvbn library specifically promises compatibility with v4.4.2 of the dropbox original. If someone installs a client library with a different scoring mechanism, the divergence can cause (and has caused) problems.

Based on dropbox/zxcvbn#290 (comment) it looks like zxcvbn-ts prefers to support new features rather than promise compatibility. That's a reasonable choice, but it creates a new moving target for backend libraries that want to provide compatibility.

Until there is a backend golang port of zxcvbn-ts, I don't think I can officially recommend it. Cross-platform compatibility is the basis for this guide.

@cainlevy cainlevy closed this Jun 11, 2021
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