Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

Password validation fix #148

Merged

Conversation

UGisBusy
Copy link
Collaborator

@UGisBusy UGisBusy commented Jan 7, 2024

Ex. Resolves #147

Changes

@$!%*?&[],.(){}":;'+\-=_~ all count toward special character n

Testing Approach (if unconventional)

docker compose up --build

@UGisBusy UGisBusy self-assigned this Jan 7, 2024
@UGisBusy UGisBusy marked this pull request as ready for review January 7, 2024 21:21
NOOBDY
NOOBDY previously approved these changes Jan 7, 2024
Copy link
Contributor

@kryr34 kryr34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update: #147 (comment)

Copy link
Contributor

@Error0229 Error0229 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FINE

Copy link
Contributor

@kryr34 kryr34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@UGisBusy UGisBusy merged commit 1ac2f30 into jykuo-love-shiritori:main Jan 8, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Defective password Regex
5 participants