Opened 16 years ago
Closed 16 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 , 16 years ago
Has patch: | set |
---|
comment:2 by , 16 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:6 by , 16 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.pk
rather than.id
inget_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.