Opened 18 years ago
Closed 18 years ago
#578 closed enhancement (invalid)
[patch] automatically do MD5 in admin backend
Reported by: | Owned by: | Adrian Holovaty | |
---|---|---|---|
Component: | contrib.admin | Version: | |
Severity: | normal | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
can perhaps close #61, but that depends if this satisfies its requirements.
This patchs adds auto md5 encryption to the admin backend, it doesn't change anything else, so it's not 'good' yet, but its better.
I am lookin to implement a meta.PasswordField which would automagically do the md5crypt and display 2 password boxes through HTML.
this is a temporal workaround.
Attachments (2)
Change History (6)
Changed 18 years ago by
Attachment: | auto_admin_md5.diff added |
---|
comment:1 Changed 18 years ago by
comment:2 Changed 18 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
This isn't a great solution, because it calculates the MD5 each time the object is saved. That means, if you create an object, save it, retrieve it and save it again, it will have calculated the MD5 of the MD5.
Changed 18 years ago by
Attachment: | auto_admin_md5_fixed.diff added |
---|
comment:3 Changed 18 years ago by
Resolution: | invalid |
---|---|
Status: | closed → reopened |
OK, I think I fixed that issue.
comment:4 Changed 18 years ago by
Resolution: | → invalid |
---|---|
Status: | reopened → closed |
No, the issue isn't fixed. The following will never be true:
password_md5 == md5.new(password_md5).hexdigest()
I don't see this patch as an improvement, so I'm closing this ticket again.
BTW, this changes api.
once you create a user object through users.create.user() you need to explicitly .save() it. I think this is OK, because all throughout the api this is also the case. I think user was the only exception.
feel free to comment ;) (or flame)