David
David
@Incognitowski is this something you want to pick up again?
It seems to have some effect on existing tests: ``` Error: Tests run: 30, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.579 s
Thanks @LoonyRules. I agree - added the BUG label.
@gayanper is this only when using GraalVM?
Thanks @p4paul ! I could merge this, but the way it's written kind of forces you to do this for each handler where you use the rate limiter. We could...
Could do ```kotlin object RateLimitUtil { var responseHeaderFunction = { ctx: Context, timeUnit: TimeUnit, numRequests: Int -> } // signature could be different } ``` Then we'd call it for...
> I like the lambda implementation for setting the headers. Feel free to discard or edit this PR as needed. I'd be happy to assist if you'd like to give...
> How important is maintaining Servlet support in this effort? I would say it's required. Personally, I am quite married to Jetty, but we have a lot of people who...
Ripped out the handler-wrapper stuff and changed to ee10. Only resource handler isn't compiling, but many tests will fail once we get it compiling.