Skip to content

Commit a123582

Browse files
SecuritySettingsSource license.self_generated: trial (#38233) (#38397)
Authn is enabled only if `license_type` is non `basic`, but `basic` is what the `LicenseService` generates implicitly. This commit explicitly sets license type to `trial`, which allows for authn, in the `SecuritySettingsSource` which is the settings configuration parameter for `InternalTestCluster`s. The real problem, that had created tests failures like #31028 and #32685, is that the check `licenseState.isAuthAllowed()` can change sporadically. If it were to return `true` or `false` during the whole test there would be no problem. The problem manifests when it turns from `true` to `false` right before `Realms.asList()`. There are other license checks before this one (request filter, token service, etc) that would not cause a problem if they would suddenly see the check as `false`. But switching to `false` before `Realms.asList()` makes it appear that no installed realms could have handled the authn token which is an authentication error, as can be seen in the failing tests. Closes #31028 #32685
1 parent 2e7b7a8 commit a123582

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

x-pack/plugin/security/src/test/java/org/elasticsearch/test/SecuritySettingsSource.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,8 @@ public Settings nodeSettings(int nodeOrdinal) {
136136
.put("xpack.security.authc.realms.file.type", FileRealmSettings.TYPE)
137137
.put("xpack.security.authc.realms.file.order", 0)
138138
.put("xpack.security.authc.realms.index.type", NativeRealmSettings.TYPE)
139-
.put("xpack.security.authc.realms.index.order", "1");
139+
.put("xpack.security.authc.realms.index.order", "1")
140+
.put("xpack.license.self_generated.type", "trial");
140141
addNodeSSLSettings(builder);
141142
return builder.build();
142143
}

0 commit comments

Comments
 (0)