Changes between Initial Version and Version 1 of Ticket #28379, comment 5
- Timestamp:
- Jul 11, 2017, 12:55:31 PM (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #28379, comment 5
initial v1 1 1 Thanks 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). 2 2 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 AccessMixinwithout meeting the conditions. Redirecting an authenticated user to the login page does not make any sense.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. 4 4