[fix][broker] Invoke custom BrokerInterceptor's onFilter method if it's defined

[jiangpengcheng]

Fixes #23670

Main Issue: #xyz

PIP: #xyz

Motivation

The BrokerInterceptor interface has a onFilter method, but it is never invoked

Modifications

Verifying this change

• Make sure that the change passes the CI checks.

• This change is already covered by existing tests, such as (please describe tests).

  • TestBrokerInterceptors.java#test

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository: jiangpengcheng#36

[jiangpengcheng]

/pulsarbot rerun-failure-checks

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.44%. Comparing base (bbc6224) to head (1ff6d0e).
Report is 775 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23676      +/-   ##
============================================
+ Coverage     73.57%   74.44%   +0.87%     
- Complexity    32624    35063    +2439     
============================================
  Files          1877     1944      +67     
  Lines        139502   147341    +7839     
  Branches      15299    16260     +961     
============================================
+ Hits         102638   109694    +7056     
- Misses        28908    29201     +293     
- Partials       7956     8446     +490     

Flag	Coverage Δ
inttests	27.42% <0.00%> (+2.84%)	⬆️
systests	24.41% <0.00%> (+0.09%)	⬆️
unittests	73.82% <100.00%> (+0.97%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...er/intercept/BrokerInterceptorWithClassLoader.java	88.07% <100.00%> (-0.99%)	⬇️
.../java/org/apache/pulsar/broker/web/WebService.java	90.80% <100.00%> (-3.35%)	⬇️

... and 665 files with indirect coverage changes