Small fix for post-login redirect#32
Conversation
…a session with no redirect_url in it.
|
We have the same problem, this fix would be great. Traceback (most recent call last): File "/usr/local/lib/python2.6/dist-packages/django/core/handlers/base.py", line 117, in get_response File "/usr/local/lib/python2.6/dist-packages/django/core/handlers/base.py", line 111, in get_response File "/usr/local/lib/python2.6/dist-packages/la_facebook/views.py", line 54, in facebook_callback File "/usr/local/lib/python2.6/dist-packages/la_facebook/callbacks/base.py", line 46, in call File "/usr/local/lib/python2.6/dist-packages/la_facebook/callbacks/base.py", line 76, in redirect_url File "/usr/lib/python2.6/urlparse.py", line 129, in urlparse File "/usr/lib/python2.6/urlparse.py", line 168, in urlsplit AttributeError: 'NoneType' object has no attribute 'find' |
|
I had the same problem and applied this fix. |
|
Please apply this pull request. We've run it in production for august, with no issues. It's a little annoying to have to reference a personal git account instead of an official release.
|
Don't break if there's no redirect URL in the callback, but there is a session with no redirect_url in it.
This happens if you've got sessions configured, and you link to the facebook_login view without providing a ?next= querystring parameter.