search

vecnet / dl-discovery

Map based discovery interface for digital library records
Other
4 stars 3 forks source link

Add 401, 403 error handler #77

Closed dbrower closed 9 years ago

dbrower commented 9 years ago

If there is no current_user it should offer a button to take the user to the log in page (similar to how the production app works).

dbrower commented 9 years ago

Need a rescue_from for the exception raised here: https://github.com/vecnet/dl-discovery/blob/qa-deploy/app/constraints/show_access_controls.rb#L20

Maybe change the name of the exception?

stevenvandervalk commented 9 years ago

Can handle the :access_denied exception in controller but perhaps you want something more like this https://github.com/richpeck/exception_handler?

Or just edit the 401 403 500 html pages to match the layout plus text in existing DL? (Currently uses a generator though...)

stevenvandervalk commented 9 years ago

Added a rescue_from handle to redirect to root_url from canonical Blacklight. a55d0efde259bd53110e0918a9f3b5be17dffae3

edited the plain html public/ error pages to fit in and added the shared header. 1164f975d9dddfedc7e8c7ba0d88fa1f893a75d1