Changes between Initial Version and Version 1 of Ticket #28379, comment 5


Ignore:
Timestamp:
Jul 11, 2017, 12:55:31 PM (7 years ago)
Author:
Dylan Verheul

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #28379, comment 5

    initial v1  
    11Thanks for clarifying Tim. I checked `django-braces`, no issues filed there as far as I can see. Of course, core Django has a much wider audience that django-braces ever had (with all due respect, former `django-braces` user here, very happy to see their ideas pulled into the main project).
    22
    3 Fact is that the default settings of Django now result in an infinite redirect to the login view if an authenticated user hits an AccessMixin and fails the conditions. That should be fixed at least. If changing the default value is too big a change, I'd vote to at least raise Permissiondenied when an authenticated user hits AccessMixin without meeting the conditions. Redirecting an authenticated user to the login page does not make any sense.
     3Fact is that the default settings of Django now result in an infinite redirect to the login view if an authenticated user hits an `AccessMixin` and fails the conditions. That should be fixed at least. If changing the default value is too big a change, I'd vote to at least raise `PermissionDenied` when an authenticated user hits `AccessMixin` without meeting the conditions. Redirecting an authenticated user to the login page does not make any sense.
    44
Back to Top