Merge pull request #4591 from abstractj/KEYCLOAK-5717

KEYCLOAK-5717
This commit is contained in:
Bill Burke 2017-10-24 17:38:28 -04:00 committed by GitHub
commit 50ccb5e5f6
5 changed files with 37 additions and 1 deletions

View file

@ -21,6 +21,10 @@ import org.keycloak.migration.ModelVersion;
import org.keycloak.models.KeycloakSession;
import org.keycloak.models.utils.DefaultKeyProviders;
import java.util.Collections;
import java.util.HashMap;
import java.util.Map;
/**
* @author <a href="mailto:mposolda@redhat.com">Marek Posolda</a>
*/
@ -31,7 +35,17 @@ public class MigrateTo3_4_0 implements Migration {
@Override
public void migrate(KeycloakSession session) {
session.realms().getRealms().stream().forEach(
r -> DefaultKeyProviders.createAesProvider(r)
r -> {
DefaultKeyProviders.createAesProvider(r);
Map<String, String> securityHeaders = r.getBrowserSecurityHeaders();
if (securityHeaders != null) {
Map<String, String> browserSecurityHeaders = new HashMap<>(securityHeaders);
browserSecurityHeaders.put("strictTransportSecurity", "max-age=31536000; includeSubDomains");
r.setBrowserSecurityHeaders(Collections.unmodifiableMap(browserSecurityHeaders));
}
}
);
}

View file

@ -36,6 +36,7 @@ public class BrowserSecurityHeaders {
headerMap.put("xContentTypeOptions", "X-Content-Type-Options");
headerMap.put("xRobotsTag", "X-Robots-Tag");
headerMap.put("xXSSProtection", "X-XSS-Protection");
headerMap.put("strictTransportSecurity", "Strict-Transport-Security");
Map<String, String> dh = new HashMap<>();
dh.put("xFrameOptions", "SAMEORIGIN");
@ -43,6 +44,7 @@ public class BrowserSecurityHeaders {
dh.put("xContentTypeOptions", "nosniff");
dh.put("xRobotsTag", "none");
dh.put("xXSSProtection", "1; mode=block");
dh.put("strictTransportSecurity", "max-age=31536000; includeSubDomains");
defaultHeaders = Collections.unmodifiableMap(dh);
headerAttributeMap = Collections.unmodifiableMap(headerMap);

View file

@ -131,6 +131,7 @@ public class MigrationTest extends AbstractKeycloakTest {
testMigrationTo3_0_0();
testMigrationTo3_2_0();
testMigrationTo3_3_0();
testMigrationTo3_4_0();
}
@Test
@Migration(versionFrom = "2.2.1.Final")
@ -218,6 +219,16 @@ public class MigrationTest extends AbstractKeycloakTest {
}
}
private void testMigrationTo3_4_0() {
Map<String, String> securityHeaders = masterRealm.toRepresentation().getBrowserSecurityHeaders();
if (securityHeaders != null) {
assertEquals("max-age=31536000; includeSubDomains",
securityHeaders.get("strictTransportSecurity"));
} else {
fail("Browser security headers not found");
}
}
private void testDockerAuthenticationFlow(RealmResource... realms) {
for (RealmResource realm : realms) {
AuthenticationFlowRepresentation flow = null;

View file

@ -132,6 +132,8 @@ robots-tag=X-Robots-Tag
robots-tag-tooltip=Prevent pages from appearing in search engines (click label for more information)
x-xss-protection=X-XSS-Protection
x-xss-protection-tooltip=This header configures the Cross-site scripting (XSS) filter in your browser. Using the default behavior, the browser will prevent rendering of the page when a XSS attack is detected (click label for more information)
strict-transport-security=HTTP Strict Transport Security (HSTS)
strict-transport-security-tooltip=The Strict-Transport-Security HTTP header tells browsers to always use HTTPS. Once a browser sees this header, it will only visit the site over HTTPS for the time specified (1 year) at max-age, including the subdomains.
permanent-lockout=Permanent Lockout
permanent-lockout.tooltip=Lock the user permanently when the user exceeds the maximum login failures.
max-login-failures=Max Login Failures

View file

@ -43,6 +43,13 @@
</div>
<kc-tooltip>{{:: 'x-xss-protection-tooltip' | translate}}</kc-tooltip>
</div>
<div class="form-group">
<label class="col-md-2 control-label" for="strictTransportSecurity"><a href="https://www.owasp.org/index.php/OWASP_Secure_Headers_Project#hsts" target="_blank">{{:: 'strict-transport-security' | translate}}</a></label>
<div class="col-sm-6">
<input class="form-control" id="strictTransportSecurity" type="text" ng-model="realm.browserSecurityHeaders.strictTransportSecurity">
</div>
<kc-tooltip>{{:: 'strict-transport-security-tooltip' | translate}}</kc-tooltip>
</div>
</fieldset>
<div class="form-group" data-ng-show="access.manageRealm">
<div class="col-md-10 col-md-offset-2">