search

metosin / ring-swagger

Swagger Spec for Clojure Web Apps
http://metosin.github.io/ring-swagger/doc/
371 stars 84 forks source link

Remove import of ring middleware from upload namespace #132

Closed borkdude closed 6 years ago

borkdude commented 6 years ago

I'm using Yada and Yada uses ring-swagger. Yada does not require the ring.middleware.multipart-params namespace, but this is required from the ring.swagger.upload namespace and therefore also introduces issue https://github.com/ring-clojure/ring/issues/251 to Yada users when they require ring.swagger.upload.

I think this bit could be moved elsewhere? https://github.com/metosin/ring-swagger/blob/master/src/ring/swagger/upload.clj#L8-L11

ikitommi commented 6 years ago

The imports can be removed. Would you like to do a PR?