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

Merge upstream master into master #7

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

jnoordsij
Copy link
Member

@jnoordsij jnoordsij commented Sep 28, 2023

ToDo:

  • Fix merge conflicts
  • Review changes

strnk and others added 16 commits December 7, 2021 15:44
listen.base-url parameter to support reverse-proxy setups
* add multi-platform build
* disable-386-build

* update versison go-release-action
* fix reuse argument multiple times

* replace whitespace with tab
Update current project status
This pull request corrects the regular expression used for username validation to correctly recognize hyphens (-).

Changes Made:
Changed the regex pattern from ^([a-zA-Z0-9_.-@])+$ to ^([a-zA-Z0-9_.\-@])+$.

Reason for Change:
In the previous regex, the hyphen (-) within the character class was interpreted as a range operator, not as a literal character. This caused usernames with hyphens to be incorrectly marked as invalid.
By escaping the hyphen (\-), the regex now correctly recognizes it as a literal character. This ensures that usernames containing hyphens are validated properly.

Points of Verification:
Confirmed that usernames containing hyphens are now correctly recognized and pass the validation.
Verified that other characters (letters, numbers, underscores, dots, and at signs) are still being properly validated.
Signed-off-by: Dmitry Shurupov <[email protected]>
Fix username validation regex to correctly recognize hyphen (-)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants