django-lazysignup icon indicating copy to clipboard operation
django-lazysignup copied to clipboard

Published 20 hours ago • verified publisher icon danfairs

authenticate passing the current request to the backends that need it

Open nschlemm opened this issue 6 years ago • 3 comments
trafficstars

e.g. https://django-axes.readthedocs.io/en/latest/3_usage.html#authenticating-users

Otherwise that will fail with AxesBackendRequestParameterRequired('AxesBackend requires a request as an argument to authenticate')

nschlemm avatar Jun 25 '19 21:06 nschlemm

@danfairs can you please give me feedback on this PR? thx!

nschlemm avatar Aug 21 '19 09:08 nschlemm

Looks fine, could you add a quick test to make sure that the request is being passed please? (Also, can I confirm that the tests don't pass with this change and the messages app added? I'm afraid I don't have a dev env right now to check it myself). Thanks!

danfairs avatar Aug 22 '19 13:08 danfairs

Just a heads-up: The changes in this PR are partly covered by #67, which was merged in the meantime.

bittner avatar Jan 25 '23 03:01 bittner