hyperfiddle

insano10 2025-04-09T19:35:45.619169Z

Does Electric v2 work ok with Jetty 12? I'm struggling to convert the https://github.com/hyperfiddle/electric-fiddle/blob/01b49391e38d2684a7cbf7f82fa58da503af5be8/src/electric_fiddle/server_jetty.clj to use the Jetty 12 classes to configure the websocket config. I'd love a hand if anyone has been here before 🙏

Dustin Getz (Hyperfiddle) 2025-04-11T13:02:25.473159Z

just adding a link - https://github.com/sunng87/ring-jetty9-adapter/commits/master/ as of April 7, 3 days ago. Note, this is a Jetty12 adapter despite the very confusing name

Dustin Getz (Hyperfiddle) 2025-04-09T19:43:18.205449Z

can you give me the background - there are so many versions to keep track of so help me load context

Dustin Getz (Hyperfiddle) 2025-04-09T19:43:37.288579Z

what were you on before, what happened, i assume you are upgrading now, why now, which version did we support

insano10 2025-04-09T19:44:59.110529Z

Electric is version v2-alpha-540-ga4699532, being used with ring 1.13.0. I am now updating ring to 1.14.1 in order to pull in an updated Jetty (`12.0.18`) to fix security vulnerabilities.

insano10 2025-04-09T19:45:35.257079Z

So Jetty is changing from v11->12 and all the classes changed 😅

Dustin Getz (Hyperfiddle) 2025-04-09T19:46:03.860549Z

in v2 our ring adapter targetted jetty 11 transitively via ring 1.13 ?

insano10 2025-04-09T19:46:18.188799Z

I believe so yes

Dustin Getz (Hyperfiddle) 2025-04-09T19:46:42.589019Z

is this due to Jetty 11 EOL announcement

insano10 2025-04-09T19:47:18.035369Z

Not specifically. Initially triggered by some dependabot security vulnerabilities flagged

insano10 2025-04-09T19:47:29.963259Z

But that would also be a good reason to upgrade

Dustin Getz (Hyperfiddle) 2025-04-09T19:48:41.242369Z

i need to talk to the team, i don't think we've done this yet for electric v3 even, the jetty 11 EOL announcement was very sudden

insano10 2025-04-09T19:49:33.704639Z

Ok great thanks. So does that mean at the moment, it's unlikely v2 works with jetty 12?

Dustin Getz (Hyperfiddle) 2025-04-09T19:49:45.305009Z

I see https://github.com/ring-clojure/ring/blob/master/CHANGELOG.md#1140-2025-03-25 was released two weeks ago

insano10 2025-04-09T19:50:24.806489Z

Oh ha I didn't realise it was so recent

Dustin Getz (Hyperfiddle) 2025-04-09T19:50:55.462799Z

I dont know if we support it, iiuc we depend on ring's API so if ring didn't change maybe it is fine?

Dustin Getz (Hyperfiddle) 2025-04-09T19:51:37.267949Z

i also have the v3 ring adapters in mind, the v2 ones may be different

insano10 2025-04-09T19:52:57.847279Z

Ok, I'll carry on seeing if I can work out how to configure the websocket config and see where I get to. Thanks

Dustin Getz (Hyperfiddle) 2025-04-09T19:54:01.625819Z

We will log a ticket internally, but it may be ... several weeks ... before we get to it

insano10 2025-04-09T19:56:21.258839Z

No problem