David
David
This has been reverted, but request handling has been rewritten so it doesn't use any futures unless the user had explicitly set a future. Stack traces should be short again!
I'll have another look in the weekend :)
@Incognitowski do you still want this merged?
We'll just leave it open then, some of Javalin's best PRs have taken months to merge 😄
I see that Ktor is using "host", but I'm having some trouble understanding why. Why is that better than "origin" ? Edit: What do other frameworks use? I believe there...
> We might have to rewrite the matching logic as you currently have to use reflectClientOrigin to work with browsers correctly, as far as I can tell from looking at...
Oh, I totally missed that ! I guess a rewrite of this plugin was long overdue 😄
@Playacem is this PR finished now?
No rush, will wait for this one. Ping me when it's ready for review :)
No, Java 17 was primarily relevant for if we were doing a Java rewrite. Since we are not, it's better that we stay on 11 (besides, there's nothing all that...