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

✨ Chore: add reserved username/password list #210

Merged
merged 4 commits into from
Jul 27, 2024
Merged

Conversation

create-issue-branch[bot]
Copy link
Contributor

Original issue description

currently commonly used unsafe names and passwords are not being checked, this issue would add a check and a list for that check of the most common names/passwords that should be blocked from registration. There are many lists online that are very similar, i would recommend we find a free-to-use list to base our list on.

Resources:

closes #207

Copy link

changeset-bot bot commented Jul 27, 2024

🦋 Changeset detected

Latest commit: b82792d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@astrolicious/studiocms Patch
@astrolicious/studiocms-blog Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@create-issue-branch create-issue-branch bot changed the title Chore: add reserved username/password list ✨ Chore: add reserved username/password list Jul 27, 2024
@Adammatthiesen Adammatthiesen marked this pull request as ready for review July 27, 2024 07:18
@Adammatthiesen Adammatthiesen requested review from a team as code owners July 27, 2024 07:18
Copy link
Contributor

@create-issue-branch[bot], thank you for opening this pull request!

Automated deployments are currently disabled for this project due to upstream issues. Please wait for a maintainer to review your changes and queue a deployment for you.

If you have any questions, feel free to ask!

Copy link
Member

@dreyfus92 dreyfus92 left a comment

Choose a reason for hiding this comment

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

LGTM 🫡

@Adammatthiesen
Copy link
Member

!coauthor

Copy link
Contributor

Co-authored-by: Adam Matthiesen <[email protected]>
Co-authored-by: Luiz Ferraz <[email protected]>
Co-authored-by: Paul Valladares <[email protected]>

@Adammatthiesen Adammatthiesen merged commit d29bda7 into main Jul 27, 2024
4 checks passed
@Adammatthiesen Adammatthiesen deleted the issue-0207 branch July 27, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore enhancement New feature or request feat
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Chore: add reserved username/password list
3 participants