Opened 9 years ago

Closed 9 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: UI/UX:

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)

backend_user_permissions.diff (144 bytes) - added by Florian Apolloner 9 years ago.
backend_user_permissions.2.diff (7.5 KB) - added by Florian Apolloner 9 years ago.
Upps, old one was empty…
doc.diff (1.5 KB) - added by Florian Apolloner 9 years ago.
I added some docs. Please apologize my bad English.
full_diff (9.0 KB) - added by Florian Apolloner 9 years ago.
full diff including docs and tests (I hope everything works, as I am really tired now XD)
full_diff.diff (9.0 KB) - added by Florian Apolloner 9 years ago.
full diff including docs and tests (I hope everything works, as I am really tired now XD)
full_diff.2.diff (11.8 KB) - added by Florian Apolloner 9 years ago.
full diff including docs and tests (I hope everything works, as I am really tired now XD) [last one missed the tests...]

Download all attachments as: .zip

Change History (12)

Changed 9 years ago by Florian Apolloner

Changed 9 years ago by Florian Apolloner

Upps, old one was empty...

comment:1 Changed 9 years ago by Florian Apolloner

Needs documentation: set
Owner: changed from nobody to Florian Apolloner
Status: newassigned
Triage Stage: UnreviewedDesign decision needed

I'll start writing the docu, and set this to "design decision needed" for now...

comment:2 Changed 9 years ago by anonymous

Keywords: sprintsept14 added

Changed 9 years ago by Florian Apolloner

Attachment: doc.diff added

I added some docs. Please apologize my bad English.

comment:3 Changed 9 years ago by Florian Apolloner

Needs documentation: unset

Changed 9 years ago by Florian Apolloner

Attachment: full_diff added

full diff including docs and tests (I hope everything works, as I am really tired now XD)

Changed 9 years ago by Florian Apolloner

Attachment: full_diff.diff added

full diff including docs and tests (I hope everything works, as I am really tired now XD)

Changed 9 years ago by Florian Apolloner

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 Changed 9 years ago by Florian Apolloner

Owner: changed from Florian Apolloner to Jacob
Status: assignednew
Triage Stage: Design decision neededReady 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 Changed 9 years ago by Florian Apolloner

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

comment:6 Changed 9 years ago by Jacob

Resolution: fixed
Status: newclosed

Fixed in [6375].

Note: See TracTickets for help on using tickets.
Back to Top