clean up app-handler setup, add middleware for custom, nicer looking, error page

src/blarg/handler.clj

@@ -7,6 +7,7 @@
         blarg.routes.accessrules
         compojure.core)
   (:require [noir.util.middleware :as middleware]
+            [noir.response :as resp]
             [compojure.route :as route]
             [taoensso.timbre :as timbre]
             [com.postspectacular.rotor :as rotor]
@@ -17,9 +18,6 @@
   (route/resources "/")
   (route/not-found "Not Found"))
 
-(defn destroy []
-  (timbre/info "picture-gallery is shutting down"))
-
 (defn init
   "init will be called once when
    app is deployed as a servlet on
@@ -49,13 +47,21 @@
   []
   (timbre/info "blarg is shutting down..."))
 
-;;append your application routes to the all-routes vector
+(defn wrap-exceptions [app]
-(def all-routes [auth-routes home-routes posts-routes files-routes rss-routes app-routes])
+  (fn [request]
+    (try
+      (app request)
+      (catch Exception e
+        (.printStackTrace e)
+        (->> (layout/render "error.html" {:error-info e})
+             (resp/status 500))))))
 
-(def app (-> all-routes
+;;append your application routes to the all-routes vector
-             (middleware/app-handler)
+(def all-routes [auth-routes home-routes posts-routes files-routes rss-routes])
-             (middleware/wrap-access-rules {:redirect "/unauthorized"} auth-required)
+
-             ;;add your middlewares here
+(def app (middleware/app-handler
-             ))
+           (conj all-routes app-routes)
+           :middleware [wrap-exceptions]
+           :access-rules [[{:redirect "/unauthorized"} auth-required]]))
 
 (def war-handler (middleware/war-handler app))

src/blarg/views/templates/error.html

@@ -8,6 +8,8 @@
 
 <p>Sorry, an error has occurred.</p>
 
+<div>
 <pre>{{error-info}}</pre>
+</div>
 
 {% endblock %}