[Improvement-16258][Registry] Add ZK authorization yaml control & add digest auth UT

apache / dolphinscheduler

Apache DolphinScheduler is the modern data orchestration platform. Agile to create high performance workflow with low-code

https://dolphinscheduler.apache.org/

Apache License 2.0

12.88k stars 4.63k forks source link

[Improvement-16258][Registry] Add ZK authorization yaml control & add digest auth UT #16277

Open pegasas opened 4 months ago

pegasas commented 4 months ago

…t auth UT

Purpose of the pull request

fix: Improvement

Brief change log

add digest UT and small refactor on ZK registry yaml for future authentication support like https://github.com/apache/dolphinscheduler/pull/16271#discussion_r1665737502

Verify this pull request

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

pegasas commented 4 months ago

Please follow pull request notice and fix code style.

Thanks, Done.

pegasas commented 4 months ago

UT failed.

It seems ZookeeperRegistryProperties would keep digest which caused curator always use digest between multiple test. I leverage practise in jdbc-registry to autowire each ZookeeperRegistryProperties to make it right.

It works now.

sonarcloud[bot] commented 4 months ago

Quality Gate passed

Issues
24 New issues
0 Accepted issues

Measures
1 Security Hotspot
64.0% Coverage on New Code
2.0% Duplication on New Code

See analysis details on SonarCloud