Clojurians Log v2
Clojure programming
Channels
Apps
aleph 2024-02-01
Nim Sadeh 2024-02-01T23:00:51.487489Z
I wonder if I missed something in the documentation, because this was a real headscratcher:
(defn handler [_]
{:status 200
:body true
})(defn handler [_]
{:status 200
:body "true"
})
the first handler is likely throwing an exception somewhere, the ring spec says body has to implement StreamableResponseBody (https://github.com/ring-clojure/ring/blob/master/ring-core/src/ring/core/protocols.clj)
and StreamableResponseBody is not extended to Booleans there
I don't know how closely aleph hews to the ring spec
looks like aleph doesn't do the ring protocol thing at all, but also doesn't have built in handling for Boolean
Matthew Davidson 2024-02-02T07:00:41.531299Z
Aleph hews to the Ring spec for the most part, but with added support for Manifold streams where it makes sense (like for bodies in raw mode). We should probably extend that protocol too, but nobody's ever complained, so I don't know if it's actually a concern.
Anyway, @nnnsadeh, hiredman has the right idea. Raw true is not allowed, and probably causing an exception somewhere. If you're using an older Aleph, it might not be logged, but if you're using the new Aleph 0.7.x, I'd expect that be logged, and will treat the lack of logging as a bug.
Nim Sadeh 2024-02-02T18:56:33.729689Z
Thanks. I haven't configured logging in my application yet, I'll do that this weekend and see