apache / apache/pulsar

brokerClientAuthenticationPlugin param doesn't restore to default after deletion dynamic config value

Open
#16,123 2 comments 0 reactions 0 assignees View on GitHub
Stale type/bug
Dominant language
Java
Stars
15.3k
Forks
3.8k
Avg merge
1d 14h
Merged PRs (30d)
160

Description

**To Reproduce**

You can start Pulsar in docker with:

`docker run -it -p 8080:8080 apachepulsar/pulsar:2.10.0 bin/pulsar standalone`

Check the default value:

Screen Shot 2022-06-18 at 11 40 54 AM

Set new value:

Screen Shot 2022-06-18 at 11 41 49 AM

Delete dynamic config and check runtime config.

My expectation is that the value should be `org.apache.pulsar.client.impl.auth.AuthenticationDisabled` again.

But it's not:

Screen Shot 2022-06-18 at 11 42 43 AM

Broker's logs tell us that the value has been deleted.

```
### Broker logs

pulsar_1 | 2022-06-18T09:42:26,274+0000 [main-EventThread] INFO org.apache.pulsar.broker.admin.impl.BrokersBase - [null] Successfully to delete dynamic configuration brokerClientAuthenticationPlugin
pulsar_1 | 2022-06-18T09:42:26,277+0000 [main-EventThread] INFO org.eclipse.jetty.server.RequestLog - 172.18.0.1 - - [18/Jun/2022:09:42:26 +0000] "DELETE /admin/v2/brokers/configuration/brokerClientAuthenticationPlugin HTTP/1.1" 200 0 "-" "Pulsar-Java-v2.10.0" 12
```

Contributor guide

Open the contributing guide

Research direction

Reproduce the issue with the provided Pulsar 2.10.0 Docker command, then set and delete brokerClientAuthenticationPlugin through the shown broker configuration endpoint. Trace the broker's dynamic-configuration handling and compare the runtime value with the default; done means deleting the dynamic value restores org.apache.pulsar.client.impl.auth.AuthenticationDisabled.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
distributed-systems
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.