metosin / reitit

A fast data-driven routing library for Clojure/Script
https://cljdoc.org/d/metosin/reitit/
Eclipse Public License 1.0
1.42k stars 255 forks source link

Multipart middleware: remove/replace dependency on broken ring.middleware.multipart-params #609

Open allentiak opened 1 year ago

allentiak commented 1 year ago

Upstream ring.middleware.multipart-params depends on removed javax.* ns. See https://github.com/ring-clojure/ring/issues/251

Offending code: https://github.com/metosin/reitit/blob/master/modules/reitit-middleware/src/reitit/ring/middleware/multipart.clj

allentiak commented 1 year ago

@ikitommi

I'm being hit by this when trying to build a toy example based on https://github.com/metosin/reitit/tree/master/examples/ring-malli-swagger

(More details: https://github.com/allentiak/scramble/commit/603f1b96bd88446e364c0413f1d61178ac7435f1)

Is there any Clojure library that you are aware of that could replace ring.middleware.multipart-params?

allentiak commented 1 year ago

UPDATE: Forget my last comment. I realized I was just simply missing a dependency.

Just for the record, I added ring/ring-jetty-adapter and it compiled.