Django

Code

Ticket #5457 (closed: fixed)

Opened 3 years ago

Last modified 2 years ago

Move permissions checking to auth backend

Reported by: Florian Apolloner Assigned to: jacob
Milestone: Component: Contrib apps
Version: newforms-admin Keywords: backend permissions sprintsept14
Cc: django@apolloner.eu Triage Stage: Ready for checkin
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

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

backend_user_permissions.diff (144 bytes) - added by Florian Apolloner on 09/14/07 08:49:59.
backend_user_permissions.2.diff (7.5 kB) - added by Florian Apolloner on 09/14/07 08:50:41.
Upps, old one was empty…
doc.diff (1.5 kB) - added by Florian Apolloner on 09/14/07 11:51:01.
I added some docs. Please apologize my bad English.
full_diff (9.0 kB) - added by Florian Apolloner on 09/14/07 16:51:12.
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 on 09/14/07 16:51:42.
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 on 09/14/07 16:53:12.
full diff including docs and tests (I hope everything works, as I am really tired now XD) [last one missed the tests...]

Change History

09/14/07 08:49:59 changed by Florian Apolloner

  • attachment backend_user_permissions.diff added.

09/14/07 08:50:41 changed by Florian Apolloner

  • attachment backend_user_permissions.2.diff added.

Upps, old one was empty...

09/14/07 09:21:57 changed by apollo13

  • status changed from new to assigned.
  • needs_better_patch changed.
  • needs_tests changed.
  • owner changed from nobody to apollo13.
  • needs_docs set to 1.
  • stage changed from Unreviewed to Design decision needed.

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

09/14/07 11:31:48 changed by anonymous

  • keywords changed from backend permissions to backend permissions sprintsept14.

09/14/07 11:51:01 changed by Florian Apolloner

  • attachment doc.diff added.

I added some docs. Please apologize my bad English.

09/14/07 11:51:23 changed by Florian Apolloner

  • needs_docs deleted.

09/14/07 16:51:12 changed by Florian Apolloner

  • attachment full_diff added.

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

09/14/07 16:51:42 changed 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)

09/14/07 16:53:12 changed 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...]

09/14/07 17:04:59 changed by apollo13

  • owner changed from apollo13 to jacob.
  • status changed from assigned to new.
  • stage changed from Design decision needed to 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>

09/14/07 17:21:50 changed by apollo13

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

09/19/07 11:52:45 changed by jacob

  • status changed from new to closed.
  • resolution set to fixed.

Fixed in [6375].

Add/Change #5457 (Move permissions checking to auth backend)




Change Properties
Action