#4406 closed Defect (fixed)
validate username for max chars
| Reported by: | coldtrick | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Elgg 1.8.6 |
| Component: | Core | Version: | 1.8.3 |
| Severity: | minor | Keywords: | |
| Cc: | brett@… | Difficulty: | trivial |
Description (last modified by brettp)
Currently the username column has a 128 char limit. This should be enforced in validate_username. Currently a very long username is accepted, but will be truncated when inserting into the database. It will thus NOT be the expected username.
I would suggest an extra check in the validate_username to limit the username length to 128 characters.
Change History (6)
comment:1 Changed 14 months ago by brettp
- Description modified (diff)
- Difficulty set to trivial
- Milestone changed from Needs Review to Elgg 1.8.5
comment:2 Changed 13 months ago by coldtrick
comment:3 Changed 12 months ago by Jerome Bakker
- Resolution set to fixed
- Status changed from new to closed
fixes #4406: validate username for max chars
Changeset: 88f1e8307f9a83b506ecae2e013125389310fed6
comment:4 Changed 12 months ago by Cash Costello
Refs #4406 fixed grammar in error message
Changeset: 7c18f2d39699fa298227d87ac531305de296c747
comment:5 Changed 11 months ago by Jerome Bakker
fixes #4406: validate username for max chars
Changeset: 88f1e8307f9a83b506ecae2e013125389310fed6
comment:6 Changed 11 months ago by Cash Costello
Refs #4406 fixed grammar in error message
Changeset: 7c18f2d39699fa298227d87ac531305de296c747
Note: See
TracTickets for help on using
tickets.
patch supplied in https://github.com/Elgg/Elgg/pull/193