dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

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

Open pegasas opened this issue 1 year ago • 3 comments

…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

image

Pull Request Notice

Pull Request Notice

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

pegasas avatar Jul 05 '24 04:07 pegasas

Please follow pull request notice and fix code style.

Thanks, Done. image

pegasas avatar Jul 05 '24 08:07 pegasas

UT failed.

image 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.

pegasas avatar Jul 06 '24 11:07 pegasas

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

github-actions[bot] avatar Dec 19 '24 00:12 github-actions[bot]

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.

github-actions[bot] avatar Dec 27 '24 00:12 github-actions[bot]