diff options
author | Ryan Ernst <ryan@iernst.net> | 2017-01-27 10:42:45 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-01-27 10:42:45 -0800 |
commit | aad51d44ab7363e53da773afa4707e0165eebda0 (patch) | |
tree | a33f4819757474775e6ff454c3d3701b871b1232 /core/src/test/java/org/elasticsearch/node | |
parent | 0f58f3f34b997528ec7258a8254551be33941c5a (diff) |
S3 repository: Add named configurations (#22762)
* S3 repository: Add named configurations
This change implements named configurations for s3 repository as
proposed in #22520. The access/secret key secure settings which were
added in #22479 are reverted, and the only secure settings are those
with the new named configs. All other previously used settings for the
connection are deprecated.
closes #22520
Diffstat (limited to 'core/src/test/java/org/elasticsearch/node')
-rw-r--r-- | core/src/test/java/org/elasticsearch/node/internal/InternalSettingsPreparerTests.java | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/core/src/test/java/org/elasticsearch/node/internal/InternalSettingsPreparerTests.java b/core/src/test/java/org/elasticsearch/node/internal/InternalSettingsPreparerTests.java index 94b3f3737c..58b6fd3959 100644 --- a/core/src/test/java/org/elasticsearch/node/internal/InternalSettingsPreparerTests.java +++ b/core/src/test/java/org/elasticsearch/node/internal/InternalSettingsPreparerTests.java @@ -21,6 +21,10 @@ package org.elasticsearch.node.internal; import org.elasticsearch.cli.MockTerminal; import org.elasticsearch.cluster.ClusterName; +import org.elasticsearch.common.settings.MockSecureSettings; +import org.elasticsearch.common.settings.SecureSetting; +import org.elasticsearch.common.settings.SecureString; +import org.elasticsearch.common.settings.Setting; import org.elasticsearch.common.settings.Settings; import org.elasticsearch.common.settings.SettingsException; import org.elasticsearch.env.Environment; @@ -161,4 +165,13 @@ public class InternalSettingsPreparerTests extends ESTestCase { assertTrue(e.getMessage(), e.getMessage().contains(".properties")); } } + + public void testSecureSettings() { + MockSecureSettings secureSettings = new MockSecureSettings(); + secureSettings.setString("foo", "secret"); + Settings input = Settings.builder().put(baseEnvSettings).setSecureSettings(secureSettings).build(); + Environment env = InternalSettingsPreparer.prepareEnvironment(input, null); + Setting<SecureString> fakeSetting = SecureSetting.secureString("foo", null, false); + assertEquals("secret", fakeSetting.get(env.settings()).toString()); + } } |