Skip to content

Commit

Permalink
Minor code cleanup
Browse files Browse the repository at this point in the history
Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
  • Loading branch information
DarshitChanpura committed Nov 17, 2023
1 parent 09c31f5 commit e5a7321
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -217,9 +217,8 @@ static String buildSecurityConfigString() {
securityConfigLines.append("plugins.security.check_snapshot_restore_write_privileges: true\n");
securityConfigLines.append("plugins.security.restapi.roles_enabled: [\"all_access\", \"security_rest_api_access\"]\n");

Check warning on line 218 in src/main/java/org/opensearch/security/tools/democonfig/SecuritySettingsConfigurer.java

View check run for this annotation

Codecov / codecov/patch

src/main/java/org/opensearch/security/tools/democonfig/SecuritySettingsConfigurer.java#L215-L218

Added lines #L215 - L218 were not covered by tests

securityConfigLines.append("plugins.security.system_indices.enabled: true\n" + "plugins.security.system_indices.indices: [")
.append(SYSTEM_INDICES)
.append("]\n");
securityConfigLines.append("plugins.security.system_indices.enabled: true\n");
securityConfigLines.append("plugins.security.system_indices.indices: [").append(SYSTEM_INDICES).append("]\n");

Check warning on line 221 in src/main/java/org/opensearch/security/tools/democonfig/SecuritySettingsConfigurer.java

View check run for this annotation

Codecov / codecov/patch

src/main/java/org/opensearch/security/tools/democonfig/SecuritySettingsConfigurer.java#L220-L221

Added lines #L220 - L221 were not covered by tests

if (!isNetworkHostAlreadyPresent(OPENSEARCH_CONF_FILE)) {
if (cluster_mode) {
Expand Down

0 comments on commit e5a7321

Please sign in to comment.