Exception when adding postgresql query result w/ timestamp to session state about kit HOT 5 CLOSED

Ah you'd need to add readers to the cookie store implementation for that. We don't provide that by default because people may choose to use different time libraries, but I get why it's problematic... any thoughts @yogthos . Here's a sample snippet from a project I'm using. Of course you can replace tick with your choice of date-time library or just native Java.

(defn wrap-base
  [{:keys [cookie-opts]}]
  (let [{:keys [cookie-secret cookie-name cookie-default-max-age]} cookie-opts
        cookie-store (session.cookie/cookie-store {:key     (.getBytes ^String cookie-secret)
                                                   :readers {'inst                 (fn [x]
                                                                                     (some-> x (tick/parse) (tick/inst)))
                                                             'time/zoned-date-time #'tick/zoned-date-time}})]
    (fn [handler]
      (cond-> handler
              true (session/wrap-session {:store        cookie-store
                                          :cookie-name  cookie-name
                                          :cookie-attrs {:max-age cookie-default-max-age}})
              true (cookies/wrap-cookies)))))

from kit.

Another option now that I think about it is to use Java's new date/time APIs instead... maybe that would be a sane default

from kit.

We could add Java default date/time API for cookies, but does seem like a fairly specific use case. It would definitely be good to add this in documentation though.

from kit.

@nikolap thanks a bunch! I was able to get it working with that info and example.

I'll leave this open since you guys are deciding on adding a default or just noting this in docs, but feel free to close it whenever.

from kit.

Added an example under cookie session documentation.

from kit.