search

timperrett / lift-shiro

Lift Integration with the Shiro security framework
http://blog.getintheloop.eu/2011/08/23/using-apache-shiro-with-lift/
Apache License 2.0
50 stars 32 forks source link

Display Error messages on the Login page #12

Closed j3ffb closed 12 years ago

j3ffb commented 12 years ago

A small change, showing the errors on the Login page instead of the index.

I have my index page secured using HasRole as I am using ldap and not all users are allowed access. The user can be authenticated successfully, but if they are not in the role a redirect loop is caused on the index page as the error message is attempted to be displayed. The login page would be a more appropriate place to show the error message that relates to permissions.

timperrett commented 12 years ago

Seems like a reasonable update, thanks.