Opened 17 years ago
Closed 17 years ago
#5457 closed (fixed)
Move permissions checking to auth backend
| Reported by: | Florian Apolloner | Owned by: | Jacob |
|---|---|---|---|
| Component: | Contrib apps | Version: | newforms-admin |
| Severity: | Keywords: | backend permissions sprintsept14 | |
| Cc: | django@… | Triage Stage: | Ready for checkin |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The included patch moves permission checking to the auth_backends as I am using an ldap server to auth against this would become handy.
The patch is against newforms-admin, but should work in trunk too...
Attachments (6)
Change History (12)
by , 17 years ago
| Attachment: | backend_user_permissions.diff added |
|---|
by , 17 years ago
| Attachment: | backend_user_permissions.2.diff added |
|---|
comment:1 by , 17 years ago
| Needs documentation: | set |
|---|---|
| Owner: | changed from to |
| Status: | new → assigned |
| Triage Stage: | Unreviewed → Design decision needed |
I'll start writing the docu, and set this to "design decision needed" for now...
comment:2 by , 17 years ago
| Keywords: | sprintsept14 added |
|---|
comment:3 by , 17 years ago
| Needs documentation: | unset |
|---|
by , 17 years ago
full diff including docs and tests (I hope everything works, as I am really tired now XD)
by , 17 years ago
| Attachment: | full_diff.diff added |
|---|
full diff including docs and tests (I hope everything works, as I am really tired now XD)
by , 17 years ago
| Attachment: | full_diff.2.diff added |
|---|
full diff including docs and tests (I hope everything works, as I am really tired now XD) [last one missed the tests...]
comment:4 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | assigned → new |
| Triage Stage: | Design decision needed → Ready for checkin |
Assigning to jacob and switching to ready for checkin, he is going to review it this weekend :)
-- Florian Apolloner <florian at apolloner.eu>
comment:5 by , 17 years ago
The diff file includes the empty init.py but it is not getting applied during patching (and I really don't know why), sry jacob
Upps, old one was empty...