Opened 17 years ago
Closed 17 years ago
#7507 closed (duplicate)
contrib.auth login form shouldn't limit username to 30 chars
| Reported by: | Chris Beaven | Owned by: | |
|---|---|---|---|
| Component: | contrib.auth | Version: | newforms-admin |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
That's only relevant if you're using the User model, and the form shouldn't be so tightly coupled with that since alternate auth backends can be used. (I for one am encountering this as a problem).
Attachments (1)
Change History (7)
comment:1 by , 17 years ago
| Has patch: | set |
|---|
comment:2 by , 17 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:6 by , 17 years ago
| Resolution: | → duplicate |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Oops, that's a whole lot of whitespace my editor removed too.
Note that my patch also throws in a very minor fix to use
return self.user_cache.pkrather than.idinget_user_id()The only problem this could cause is if db backends fail if you try to compare a long string against a short char db field. I don't think this is the case, just noting a potential caveat. If this is the case, then the default backend could be fixed to skip any auth attempts with more than 30 char usernames.