lhotari opened a new pull request, #15637:
URL: https://github.com/apache/pulsar/pull/15637
### Motivation
Follow up on #14353
There were filter instances for each context path which made
`maxConcurrentHttpRequests` and `httpRequestsMaxPerSecond` configuration
options not work as expected. The actual limits would be multiple times higher
dependencing on the way clients access different context paths.
### Modifications
- Refactor solution to use FilterHolder for sharing Servlet Filter instances
- Fix invalid Jersey API usage
- `config.packages("jersey.config.server.provider.packages",
javaPackages);` is invalid
- The API takes a java package directly, passing
`jersey.config.server.provider.packages` doesn't make any sense.
- When configuring individual resource classes, `.register` method should
be used instead of registering the package which includes the individual
resource class.
- Don't use recursive resource class scanning since it might results in
unintended resource classes being exposed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
