search

lagotto / alm-report

ALM Reports
http://almreports.plos.org/
MIT License
8 stars 3 forks source link

Properly handle expired sessions #34

Closed mfenner closed 10 years ago

mfenner commented 10 years ago

Sessions expire after 60 min, and users can then run into this error:

RuntimeError (No DOIs saved in session!):
  app/controllers/reports_controller.rb:12:in `generate'
  app/controllers/application_controller.rb:105:in `save_session_dois' (pid:27373)

Instead of raising a RuntimeError we should handle this properly with a flash message and a redirect to the homepage.