Migrate deprecated commons-configuration 1.x to commons-configuration 2.x#4604
Merged
merlimat merged 13 commits intoapache:masterfrom May 26, 2025
Merged
Conversation
lhotari
force-pushed
the
lh-migrate-to-commons-configuration2
branch
3 times, most recently
from
f5e0d38 to
fe8c635
Compare
lhotari
force-pushed
the
lh-migrate-to-commons-configuration2
branch
from
fe8c635 to
de1f23c
Compare
This reverts commit 073c267.
lhotari
changed the title
lhotari
requested review from
StevenLuMT,
dlg99,
eolivelli,
hangc0276,
hezhangjian and
zymap
merlimat
approved these changes
May 23, 2025
- system properties are read when ServerConfiguration gets initialized
merlimat
pushed a commit
that referenced
this pull request
May 26, 2025
… 2.x (#4604) * Migration to commons-configuration2 and commons-lang3 * Change test that breaks with commons configuration 2 * Add commons-beanutils dependency * Revert "Change test that breaks with commons configuration 2" This reverts commit 073c267. * Fix ConfigKeyTest * Configure list handling to match commons-configuration 1.x default behavior * Fix the issue in using the API to configure defaults * Configure list delimiter handler also for distributedlog's ConfigurationSubscription * Address issue with system properties being null * Use list delimiter handler in DistributedLogConfiguration too * Fix spotbugs check * Fix ConfigurationTest - system properties are read when ServerConfiguration gets initialized * Fix checkstyle
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jun 30, 2025
…ns.configuration2, base on Migrate deprecated commons-configuration 1.x to commons-configuration 2.x apache/bookkeeper#4604
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jun 30, 2025
…ns.configuration2, base on Migrate deprecated commons-configuration 1.x to commons-configuration 2.x apache/bookkeeper#4604
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jun 30, 2025
…ons-configuration 2.x, base on apache/bookkeeper#4604
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jul 1, 2025
…ons-configuration 2.x, base on apache/bookkeeper#4604
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jul 2, 2025
…ons-configuration 2.x, base on apache/bookkeeper#4604
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jul 3, 2025
…ons-configuration 2.x, base on apache/bookkeeper#4604
StevenLuMT
added a commit
to StevenLuMT/pulsar
that referenced
this pull request
Jul 4, 2025
…ons-configuration 2.x, base on apache/bookkeeper#4604
priyanshu-ctds
pushed a commit
to datastax/bookkeeper
that referenced
this pull request
Jul 11, 2025
… 2.x (apache#4604) * Migration to commons-configuration2 and commons-lang3 * Change test that breaks with commons configuration 2 * Add commons-beanutils dependency * Revert "Change test that breaks with commons configuration 2" This reverts commit 073c267. * Fix ConfigKeyTest * Configure list handling to match commons-configuration 1.x default behavior * Fix the issue in using the API to configure defaults * Configure list delimiter handler also for distributedlog's ConfigurationSubscription * Address issue with system properties being null * Use list delimiter handler in DistributedLogConfiguration too * Fix spotbugs check * Fix ConfigurationTest - system properties are read when ServerConfiguration gets initialized * Fix checkstyle (cherry picked from commit 542ea09)
sandeep-ctds
pushed a commit
to datastax/bookkeeper
that referenced
this pull request
Jul 22, 2025
… 2.x (apache#4604) * Migration to commons-configuration2 and commons-lang3 * Change test that breaks with commons configuration 2 * Add commons-beanutils dependency * Revert "Change test that breaks with commons configuration 2" This reverts commit 073c267. * Fix ConfigKeyTest * Configure list handling to match commons-configuration 1.x default behavior * Fix the issue in using the API to configure defaults * Configure list delimiter handler also for distributedlog's ConfigurationSubscription * Address issue with system properties being null * Use list delimiter handler in DistributedLogConfiguration too * Fix spotbugs check * Fix ConfigurationTest - system properties are read when ServerConfiguration gets initialized * Fix checkstyle (cherry picked from commit 542ea09)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
commons-configuration 1.x is deprecated and won't be updated. There's a low severity CVE-2025-46392 in commons-configuration 1.x.
This PR migrates the commons-configuration 1.x usage in BookKeeper to use commons-configuration 2.x
There's also changes to migrate from commons-lang to commons-lang3.