Page MenuHomePhabricator

Closed, ResolvedPublic


AntiSpoof_body.php has

$deprecated = array_search( "SCRIPT_DEPRECTED", $testScripts );

I think the typo means usernames with characters in the ranges CJK Radicals Supplement, Kangxi Radicals, CJK Compatibility Ideographs, etc. will not be rejected.

But I tried to add a failing test for this to SpoofTest.php and it passes, so I can't demonstrate the failure.

Version: master
Severity: minor



Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:52 AM
bzimport set Reference to bz40649.

Created attachment 11152
Changes to SpoofTest.php

git review fails, maybe I don't have rights. Here's a patch that introduces testCheckBadChars() with some working tests, but the tests for SCRIPT_DEPRECATED glyphs doesn't work.


Gerrit change 25856

(if you need to get it and improve it, use gerrit -d 25856)

The first time you clone a repository, you can use git review -s to add git review and Gerrit specific hooks. One of them will add the last git Change-Id to the commit message.

If you commit stuff before the git review -s, then launch git review, it will do that for you (Creating a git remote called "gerrit" that maps to: ssh://<your username>

But it won't edit your commit message, so Gerrit will reject it with this message:
remote: ERROR: missing Change-Id in commit message

It will even suggest you a commit message (your commit message + the Change-Id).

remote: Change-Id: I98e4bc5aa0034369910effda73625ce8b03288be

What you have to do is:
(1) copy/paste this line (without the remote: )
(2) git commit --amend
(3) add this line to your commit message
(4) git review

Patch in Gerrit needs an update.

Change 109464 had a related patch set uploaded by Arav93:
Renamed SCRIPT_DEPRECTED to SCRIPT_DEPRECATED in Antispoof_body.php

Change 109464 merged by jenkins-bot:
Renamed SCRIPT_DEPRECTED to SCRIPT_DEPRECATED in Antispoof_body.php

Should be fixed by the patch above.