account - 404 when going to a @login_required page

[tubaman]

When using the account starter app, if I hit a @login_required page I get a 404. It looks like it's trying to redirect me to /accounts/login/ which doesn't exist. Here's how to recreate the bug:

1. mkvirtualenv testloginurl

2. pip install pinax-cli

3. pinax start account testloginurl

4. cd testloginurl

5. pip install -r requirements.txt

6. ./manage.py migrate

7. ./manage.py loaddata sites

8. add this import to testloginurl/urls.py: from django.contrib.auth.decorators import login_required

9. add this line to urlpatterns in testloginurl/urls.py: url(r"^protected$", login_required(TemplateView.as_view(template_name="homepage.html")), name="protected"),

10. ./manage.py runserver

11. Go to http://localhost:8000/protected <- See the 404

[tubaman]

Here's a workaround but it's probably not the right fix.

[tubaman]

Ok, it seems I need to use account.decorators.login_required instead of django.contrib.auth.decorators.login_required. I figured this out by digging through the code. I couldn't find it in the docs

[tubaman]

I've create a pull request against the docs. Is there a way we can get rid of this login_decorator and use Django's built-in instead?

[paltman]

I believe that @brosner's version in DUA handles some extra things like handing next but I'm not completely sure without studying the differences in code. I do know there is likely a very good reason why DUA is shipping with it's own rather than using the built-in.