search

ring-clojure / ring

Clojure HTTP server abstraction
MIT License
3.76k stars 520 forks source link

Update jetty-server #506

Closed KGOH closed 3 months ago

KGOH commented 4 months ago

Currently jetty-server has 11.0.21 version, which fixes a number of bugs https://github.com/jetty/jetty.project/releases/tag/jetty-11.0.21

ieugen commented 4 months ago

While ring will eventually pick this up, you can explicitly depend on that version of jetty in your code and it will override the version that comes with ring-jetty-adapter.

I do think ring-jetty-adapter should not ship with buggy versions of jetty, but maintenance is costly.

KGOH commented 3 months ago

ring-jetty-adapter should not ship with buggy versions of jetty

oh what a wonderful world would it be

Yeah, explicit dependency did work for me. Thanks

weavejester commented 3 months ago

From the Jetty changelog, there doesn't appear to be anything in the Jetty release that would effect Ring, but I'll release a new version anyway.