Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 1320 by josh.hamacher: Auto-added names cause exceptions.

*NOTE: Do not post confidential information in this bug report.*

What version are you running?

What's the URL of the page containing the problem?
(That is, any review page with the details described below.)

What steps will reproduce the problem?
1. Create a new review.
2. Under "People:", enter something such as "user1; user2".
3. Refresh the page.  Observe the exception.

What is the expected output? What do you see instead?
This seems like a breakdown in input verification.  A user unfamiliar with
ReviewBoard might try to use a semicolon, rather than a comma, to separate
users.  ReviewBoard helpfully adds the string as a user but other parts of
the system seem unable to handle the format.  In my opinion it's reasonable
to disallow semicolons and most other punctuation when verifying usernames.
  Or, if arbitrary characters are allowed, the rest of the system should be
able to handle them.

What operating system are you using? What browser?
Red Hat Enterprise Linux 5.3 and Firefox 3.5.3, but it shouldn't matter.

Please provide any additional information below.

You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:

You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to
To unsubscribe from this group, send email to
For more options, visit this group at

Reply via email to