sofastack / sofa-registry

SOFARegistry is a production-level, low-latency, high-availability service registry powered by Ant Financial.
https://www.sofastack.tech/sofa-registry/docs/Home
Apache License 2.0
650 stars 246 forks source link

Remove log4j2 Strings import && bump commons-lang to commons-lang3 #346

Closed aofall closed 10 months ago

aofall commented 10 months ago

Motivation:

The usage of Log4j2 Strings Util class in the code makes users force dependency on log4j2. We can use commons-lang instead or maintain self Utils of String in the project.

Modification:

Bump commons-lang to commons-lang3. And used the commons-lang3 to instead Strings usage of Log4j2 in the code.

Result:

Fixes #345 .

sofastack-cla[bot] commented 10 months ago

Hi @aofall, welcome to SOFAStack community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

aofall commented 10 months ago

I have signed the CLA but the status of the bot is still unsigned, can anyone help?

image

nobodyiam commented 9 months ago

Would you please reopen the pr so that we could look into this cla issue? cc @zhenjunMa