You're right.  We can handle this better. 

On Feb 13, 2013, at 4:40 PM, kluo <notifications@github.com> wrote:

> In main/courses/actions.py:
> 
> > @@ -31,8 +32,12 @@ def inner(request, *args, **kw):
> >  
> >          if not user.is_authenticated():
> >              messages.add_message(request,messages.ERROR, 'You must be logged-in to view the content you chose.')
> > -
> > -            return HttpResponseRedirect(reverse('courses.views.main', args=(request.common_page_data['course_prefix'], request.common_page_data['course_suffix'],)))
> > +            if settings.SITE_NAME_SHORT == "Stanford":
> > +                if course.institution_only:
> > +                    return HttpResponseRedirect(reverse('shib_login') + "?next=" + request.path)
> If shib login fails this results in a redirect loop, but I'm not sure if that ever actually happens?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>