Georgios Andrianakis
Georgios Andrianakis
I agree, this does not look correct. However `OidcSession` is part of the public API, so we can just change it. We likely need to introduce a new method and...
Thanks for this! Seems reasonable to me, but I will leave it up to @iocanel to decide
You aren't doing anything wrong :). It's just the default setting kicking on.
No problem at all! Thanks for persevering!
Thanks for looking into this @Postremus! While working on this, please make sure that the JAX-RS TCK is passing with this change
Nope, it's this one: https://github.com/quarkusio/resteasy-reactive-testsuite. Just run `mvn install` on that module once you've built your change in Quarkus.
@Postremus thanks for this. Any chance you can do a small benchmark to see if this introduces any noticeable overhead?
Thanks! We also want to make sure throughput for simple hello-world style rest calls is not affected
@Postremus are you still interested in driving this home? It seems like a rather good improvement
Does someone have a ready made sample I can use to see the problem?