[fix][broker] Fix Filter NPE

[Technoboy-]

Motivation

After merging #20422 , the broker may encounter NPE when config :

brokerInterceptors = xxx-interceptor
disableBrokerInterceptors = true

2023-07-03T10:26:00,507+0000 [main] INFO  org.apache.pulsar.broker.intercept.BrokerInterceptors - Skip loading the broker interceptors when disableBrokerInterceptors is true

See

pulsar/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/ProcessHandlerFilter.java

Lines 38 to 53 in dc25810

	public ProcessHandlerFilter(PulsarService pulsar) {
	this.interceptor = pulsar.getBrokerInterceptor();
	this.interceptorEnabled = !pulsar.getConfig().getBrokerInterceptors().isEmpty();
	}
	@Override
	public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain)
	throws IOException, ServletException {
	if (interceptorEnabled
	&& !StringUtils.containsIgnoreCase(request.getContentType(), MediaType.MULTIPART_FORM_DATA)
	&& !StringUtils.containsIgnoreCase(request.getContentType(), MediaType.APPLICATION_OCTET_STREAM)) {
	interceptor.onFilter(request, response, chain);
	} else {
	chain.doFilter(request, response);
	}
	}

pulsar/pulsar-broker/src/main/java/org/apache/pulsar/broker/intercept/BrokerInterceptors.java

Lines 60 to 64 in dc25810

	public static BrokerInterceptor load(ServiceConfiguration conf) throws IOException {
	if (conf.isDisableBrokerInterceptors()) {
	log.info("Skip loading the broker interceptors when disableBrokerInterceptors is true");
	return null;
	}

The master branch has fixed this by #19376

For the judgment of interceptors, we need to be consistent with the master, that is, to use the disableBrokerInterceptors judgment first

Documentation

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