Skip to content

Conversation

simplyNour
Copy link
Contributor

@simplyNour simplyNour commented Oct 13, 2025

The PR fixes the premature regex check for usernames on Topcoder to allow multiple occurrences of underscores and alphanumeric characters.
Fixes: #2669

Copy link
Contributor

Automatic validation of changes

Target F+ Check F- Check
Topcoder ✔️   Pass ✔️   Pass

@ppfeister
Copy link
Member

Were the F+s due to single-letter handles? Since that seems to be the main change, here.

If so, would it be worth specifying a minimum username length? {3,} instead of + for min 3, for instance.

@akh7177
Copy link
Contributor

akh7177 commented Oct 14, 2025

Were the F+s due to single-letter handles? Since that seems to be the main change, here.

If so, would it be worth specifying a minimum username length? {3,} instead of + for min 3, for instance.

According to what I observed, even without any changes TopCoder was working correctly, without any F+ or F-

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.

False positive for: Topcoder

3 participants