hyperfiddle 2025-04-09

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

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.

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 ?

I believe so yes

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

is this due to Jetty 11 EOL announcement

Not specifically. Initially triggered by some dependabot security vulnerabilities flagged

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

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

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

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