Skip to content

Commit

Permalink
Merge pull request #46271 from Ladicek/maven-fix-discovering-settings…
Browse files Browse the repository at this point in the history
…-security-xml

Maven Resolver: fix discovering settings-security.xml
  • Loading branch information
aloubyansky authored Feb 14, 2025
2 parents 9fcb390 + b435ed7 commit 6c4d7b2
Showing 1 changed file with 11 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -533,7 +533,18 @@ private DefaultRepositorySystemSession newRepositorySystemSession() throws Boots
DefaultSettingsDecryptionRequest decrypt = new DefaultSettingsDecryptionRequest();
decrypt.setProxies(settings.getProxies());
decrypt.setServers(settings.getServers());
// need to set `settings-security.xml` location extra, because it isn't discovered
// by BeanBag when constructing `DefaultSecDispatcher`
File settingsSecurityXml = null;
boolean setSettingsSecurity = !System.getProperties().containsKey(SETTINGS_SECURITY)
&& (settingsSecurityXml = new File(getUserMavenConfigurationHome(), "settings-security.xml")).exists();
if (setSettingsSecurity) {
System.setProperty(SETTINGS_SECURITY, settingsSecurityXml.toString());
}
SettingsDecryptionResult decrypted = getSettingsDecrypter().decrypt(decrypt);
if (setSettingsSecurity) {
System.clearProperty(SETTINGS_SECURITY);
}

if (!decrypted.getProblems().isEmpty() && log.isDebugEnabled()) {
for (SettingsProblem problem : decrypted.getProblems()) {
Expand Down

0 comments on commit 6c4d7b2

Please sign in to comment.