OpenFn / OpenFn/lightning

Make user passwords more secure

Open
#3,012 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

needs detail needs technical design
Dominant language
Elixir
Stars
296
Forks
86
Avg merge
1d 13h
Merged PRs (30d)
50

Description

Our current password validation rule only enforces minimum length. It does not require users to set a stronger password by requiring at least one lower case, one upper case, one number, and an alphanumeric character in the password.

On the marketing site, we enforce this requirement for users creating a new account and we would love this to be implemented on the application for when a user is signing up via the/register route, resetting their password, and updating their password.

See the current implementation here:
https://github.com/OpenFn/lightning/blob/8db08f8c13bf2354c8f0037b98a0865ab3fbdc68/lib/lightning/accounts/user.ex#L194

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the password validation implementation in lib/lightning/accounts/user.ex around line 194. Trace how it is used by the /register route, password reset, and password update flows. Done means all three flows enforce the stated lowercase, uppercase, number, and alphanumeric requirements.

Written by the indexing model from the issue text.

Assessment

Tech stack
elixir
Domain
authentication, security
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.