From 657ba2bacc1ff42ee574514ed10080961e921d1d Mon Sep 17 00:00:00 2001 From: Prarthona Paul Date: Mon, 19 Aug 2024 12:28:11 -0400 Subject: [PATCH 1/3] Bump enytron client schema from 1.7 to 1.8 --- .../resources/schema/elytron-client-1_8.xsd | 1395 +++++++++++++++++ 1 file changed, 1395 insertions(+) create mode 100644 auth/client/src/main/resources/schema/elytron-client-1_8.xsd diff --git a/auth/client/src/main/resources/schema/elytron-client-1_8.xsd b/auth/client/src/main/resources/schema/elytron-client-1_8.xsd new file mode 100644 index 00000000000..104fecbb268 --- /dev/null +++ b/auth/client/src/main/resources/schema/elytron-client-1_8.xsd @@ -0,0 +1,1395 @@ + + + + + + + + + + + + + + Authentication configuration for outbound connections and SSL configuration for outbound connections. + + + + + + + + + + + + + + + + + + Complex type to contain SSLContext definitions that can subsequently be matched by the ssl context rules. + + + + + + + + The default SSLContext. + + + + + + + + + + SSLContext definition. + + + + + + + + + + Configuration to filter the enabled cipher suites. + + + + + + + Define a space separated list of SSLContext protocols to be supported. + + + + + + + Once the available providers have been identified only the provider with the name defined on this element will be used. + + + + + + + + + + + + Name used for referencing SSLContext from other parts of the configuration. + + + + + + + + + Rules used to determine which SSLContext should be used for individual resources. + + + + + + + + + + + + + + Matches the protocol within the URI against the name specified in this match element. + + + + + + + Matches the host name from within the URI against the name specified in this match element. + + + + + + + Matches the path from the URI against the name specified in this match element. + + + + + + + Matches the port number specified within the URI against the number in this match element. + This only matches against the number specified within the URI and not against any default derived from the protocol. + + + + + + + Matches the scheme specific part of the URI against the name specified within this element. + + + + + + + Matches where the protocol of the URI is 'domain' and the scheme specific part of the URI is the name specified within this match element. + + + + + + + + + + + + User info can be embedded within a URI, this rule matches when there is no user-info. + + + + + + + Matches when the user-info embedded in the URI matches the name specified within this element. + + + + + + + + Definition of rule for assigning SSLContexts to resources. + + + + + + + + A name of SSLContext, which should be used if given rule match. + + + + + + + + + + + Named configurations to be referenced from authentication rules. + + + + + + + + + + + A configuration that can then be matched from the authentication rules. + + + + + + + DEPRECATED. Override the destination host name for the authenticated call. + + + + + + + DEPRECATED. Override the destination port number for the authenticated call. + + + + + + + DEPRECATED. Override the protocol for the authenticated call. + + + + + + + + The name of the realm that will be selected by the SASL mechanism if required. + + + + + + + A regular expression pattern and replacement to re-write the user name used for authentication. + + + + + + + A SASL mechanism selector using the syntax from org.wildfly.security.sasl.SaslMechanismSelector,fromString(). + + + + + + + One or more properties to be passed to the authentication mechanisms. + + + + + + + + The name that should be used for authorization if different from the authentication identity. + + + + + + + + + + + Name used for referencing the authentication configuration. + + + + + + + + + One or more credentials to be assembled into a protection parameter when initialising the credential store. + + + + + + + + + + + + + + A pem encoded public key. + + + + + + + + + DEPRECATED. Credential obtained from local kerberos ticket cache. + + + + + + + + + + The location of a private ssh key to be used as credentials. + + + + + + + + + + + + The directory containing the SSH key file. Set to "[user.home]/.ssh" by default + + + + + + + The name of the private key file. Set to "id_rsa", "id_dsa, and "id_ecdsa" by default. + + + + + + + The name of the file containing the known SSH hosts. Set to "known_hosts" by default. + + + + + + + + + A public and private key pair. + + + + + + + A pem encoded public key. + + + + + + + A pem encoded private key. + + + + + + + A PEM encoded private key in OpenSSH format. The public key to be used will be obtained from the encoded private key + + + + + + + + + + A PEM encoded private key in OpenSSH format. The public key to be used will be obtained from the encoded private key + + + + + + + + + + + + The PEM encoded private key in OpenSSH format. + + + + + + + + + A pem encoded private key and corresponding certificate. + + + + + + + A pem encoded private key. + + + + + + + Corresponding certificate. + + + + + + + + + + Rules to be applied for outbound connections to match against an appropriate authentication configuration. + + + + + + + + + + + Authentication client rule definition. + + + + + + + + A configuration to use for the rule. + + + + + + + + + + + + The user name to use for authentication. + + + + + + + Switch to anonymous authentication. + + + + + + + + Define how java.security.Provider instances are located when required. + + + + + + + The providers from java.security.Security.getProviders() + + + + + + + Providers loaded using service loader discovery from the module specified, + if no module is specified the ClassLoader which loaded the authentication client is used. + + + + + + + + + + Define how the Webservices client will authenticate. + + + + + + + WS client will use the credentials with this HTTP mechanism. + + + + + + + WS client will use the credentials with this WS-Security authentication type. + + + + + + + + + + + The java.security.Provider instances either inherited or defined in this configuration will be used to locate the available SASL client factories. + + + + + + + SASL client factories will be discovered using service loader discovery on the specified module or if not specified using the ClassLoader loading the configuration. + + + + + + + + The presence of this element enables checking the peer's certificate against multiple certificate revocation lists. + + + + + + + + + + + The presence of this element enables checking the peer's certificate against a certificate revocation list. + + + + + + + URI of certificate revocation list file. Alternative to "path" and "resource". + + + + + + + The module resource to use as certificate revocation list. Alternative to "path" and "uri". + + + + + + + + Path to the certificate revocation list. Alternative to "resource" and "uri". + + + + + + + + + The presence of this element enabled checking the peer's certificate against a certificate revocation list. + + + + + + + URI of certificate revocation list file. Alternative to "path" and "resource". + + + + + + + The module resource to use as certificate revocation list. Alternative to "path" and "uri". + + + + + + + + Path to the certificate revocation list. Alternative to "resource" and "uri". + + + + + + + The maximum number of non-self-issued intermediate certificates that may exist in a certification path. + This attribute is deprecated and maximum-cert-path attribute in trust-manager should be used instead. + + + + + + + + + The presence of this element enabled checking the peer's certificate using online certificate status protocol. + + + + + + OCSP responder URI to override those extracted from certificate. + + + + + + + Alias of OCSP Responder certificate. + + + + + + + Keystore for OCSP Responder certificate. trust-manager keystore is used by default and responder-certificate has to be defined. + + + + + + + Prefer certificate revocation list revocation over OCSP if certificate-revocation-list is defined. + + + + + + + + + Complex type to contain the definitions of the key stores. + + + + + + + + + + + An individual key store definition. + + + + + + + + + Load from file. + + + + + + + Load the file from the URI specified. + + + + + + + Load as a resource from the Thread context classloader. + + + + + + + + + + + + + + + + + Name used for referencing the key store. + + + + + + + The key store type. + + + + + + + The key store provider. + + + + + + + Whether to wrap keystore to allow storing two-way passwords in it. + + + + + + + + + Defines a reference to an entry within a KeyStore for an entry to use. + + + + + + + + + + + + + Name of the KeyStore being referenced to load the entry from. + + + + + + + The alias of the entry to load from the referenced KeyStore, + this can only be omitted for KeyStores that contain only a single entry. + + + + + + + + + Reference to a credential stored in a credential store. + + + + + + Credential store name. + When used, attribute "alias" need to be specified. + + + + + + + Alias in the credential store. + Ignored if "store" is not specified. + + + + + + + Credential store password in clear text. + Supersedes "store" and "alias" attributes. + + + + + + + + + Key manager definition. Provides credentials to authenticate against the peer. + + + + + + + + + + + + + A reference to a KeyStore that will be used to initialise the KeyManager. + + + + + + + The alias of the key to filter from the referenced KeyStore. + + A comma separated list of aliases or one of the following formats can be used: ALL:-alias1:-alias2, NONE:+alias1:+alias2 + + + + + + + Key manager provider name. + + + + + + + Key manager algorithm. + + + + + + + + + Trust manager definition. Decides whether credentials presented by a peer should be accepted. + + + + + + Trust manager provider name. + + + + + + + Trust manager algorithm. + + + + + + + The maximum number of non-self-issued intermediate certificates that may exist in a certification path. + + + + + + + Check revocation status only of leaf certificates. + + + + + + + Accept certificate if revocation status is unknown. + + + + + + + + + Trust store definition. + + + + + + A reference to a KeyStore that will be used to initialise the TrustManager. + + + + + + + + + A bearer token. + + + + + + A bearer token value. + + + + + + + + + An OAuth 2 bearer token. + + + + + + + + + + + + + + + An OAuth 2 bearer token client credentials. + + + + + + + + + The client identifier. + + + + + + + The client secret. + + + + + + + + + An OAuth 2 bearer token client credentials. + + + + + + + + + The client identifier. + + + + + + + + + An OAuth 2 bearer token resource owner credentials. + + + + + + + + + Name used for referencing the resource owner. + + + + + + + The resource owner password. + + + + + + + + + An OAuth 2 bearer token resource owner credentials. + + + + + + + + + Name used for referencing the resource owner. + + + + + + + + + + Complex type to contain the definitions of the credential stores. + + + + + + + + + + + An individual credential store definition. + + + + + + + + + + + + + Name used for referencing the credential store. + + + + + + + The credential store type, e.g. KeyStoreCredentialStore. + + + + + + + The name of the provider to use to instantiate the CredentialStoreSpi. + If the provider is not specified then the first provider found that can create an instance of the specified 'type' will be used. + + + + + + + + + + + Matches the abstract type and/or authority against the values specified within this match element. + + + + + + Name used for referencing the abstract type. + + + + + + + Abstract type authority. + + + + + + + + + Complex type to contain the definitions of the attributes. + + + + + + + + + + + An individual attribute definition. + + + + + + Name used for referencing the attribute. + + + + + + + The value of the attribute. + + + + + + + + + A clear password definition. + + + + + + A password specified in the clear. + + + + + + + + + A masked password definition. + + + + + + The algorithm that was used to encrypt the password. + + + + + + + The initial key material that was used to encrypt the password. + + + + + + + The iteration count that was used to encrypt the password. + + + + + + + The salt that was used to encrypt the password. + + + + + + + The base64 encrypted password. (without the "MASK-" prefix) + + + + + + + The initialization vector that was used to encrypt the password. + + + + + + + + + + + Module reference. + + + + + + Module name. + + + + + + + + + + + + + Resource reference. + + + + + + Name used for referencing the resource. + + + + + + + Module name. + + + + + + + + + + + + + + + + + + + + + + + + + Port number. + + + + + + + + + + + + + + Representation of a key/value property pair. + + + + + + + The key for this property. + + + + + + + The value for this property. + + + + + + + + + + + + A regular expression substitution type. + + + + + + A regular expression pattern. + + + + + + + A regular expression replacement used for re-write. + + + + + + + + + The format of the selector is described in Javadoc of org.wildfly.security.ssl.CipherSuiteSelector.fromString(selector). + + + + + + + + + A Uniform Resource Identifier (URI) reference. + + + + + + A Uniform Resource Identifier (URI). + + + + + + + + + Cipher suite selector type. + + + + + + The format of the selector is described in Javadoc of org.wildfly.security.ssl.CipherSuiteSelector.fromString(selector). + + + + + + + The format of the names is described in Javadoc of org.wildfly.security.ssl.CipherSuiteSelector.fromNamesString(names). + + + + + + + + + + The mechanism names the credential should be usable with. + Names will be converted to OIDs and used together with OIDs from mechanism-oids attribute. + + + + + + + The mechanism OIDs the credential should be usable with. + Will be used together with OIDs derived from names from mechanism-names attribute. + + + + + + + + A list of String. + + + + From f171c96aa426e99bcb87e086548591d329b609bd Mon Sep 17 00:00:00 2001 From: Prarthona Paul Date: Fri, 10 May 2024 12:34:09 -0400 Subject: [PATCH 2/3] ELY-434 [Preview] OCSP Stapling Support --- .../auth/client/ElytronXmlParser.java | 77 +++++++- .../resources/schema/elytron-client-1_8.xsd | 39 ++++ .../auth/client/ElytronXmlParserTest.java | 8 + .../auth/client/test-wildfly-config-v1_8.xml | 64 +++++++ .../wildfly/security/ssl/ElytronMessages.java | 3 + .../security/ssl/SSLContextBuilder.java | 128 ++++++++++++- .../ssl/X509RevocationTrustManager.java | 90 ++++++---- .../security/ssl/SSLAuthenticationTest.java | 169 +++++++++++++++--- .../ssl/ssl-authentication-config.xml | 80 ++++++++- .../ssl/test/util/CAGenerationTool.java | 8 +- 10 files changed, 604 insertions(+), 62 deletions(-) create mode 100644 auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml diff --git a/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java b/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java index 967df3725b8..039344daf48 100644 --- a/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java +++ b/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java @@ -154,7 +154,8 @@ private enum Version { VERSION_1_4("urn:elytron:client:1.4", VERSION_1_3), VERSION_1_5("urn:elytron:client:1.5", VERSION_1_4), VERSION_1_6("urn:elytron:client:1.6", VERSION_1_5), - VERSION_1_7("urn:elytron:client:1.7", VERSION_1_6); + VERSION_1_7("urn:elytron:client:1.7", VERSION_1_6), + VERSION_1_8("urn:elytron:client:1.8", VERSION_1_7); final String namespace; @@ -465,6 +466,7 @@ private static void parseSslContextType(final ConfigurationXMLStreamReader reade ExceptionSupplier trustStoreSupplier = null; DeferredSupplier providersSupplier = new DeferredSupplier<>(providers); TrustManagerBuilder trustManagerBuilder = new TrustManagerBuilder(providersSupplier, location); + boolean acceptOcspStapling = false; while (reader.hasNext()) { final int tag = reader.nextTag(); @@ -536,6 +538,13 @@ private static void parseSslContextType(final ConfigurationXMLStreamReader reade parseCertificateRevocationLists(reader, trustManagerBuilder, xmlVersion); break; } + case "accept-ocsp-stapling": { + if (isSet(foundBits, 10)) throw reader.unexpectedElement(); + foundBits = setBit(foundBits, 10); + if (!xmlVersion.isAtLeast(Version.VERSION_1_8)) throw reader.unexpectedElement(); + acceptOcspStapling = parseOcspStaplingType(reader, trustManagerBuilder, xmlVersion, keyStoresMap); + break; + } default: throw reader.unexpectedElement(); } } else if (tag != END_ELEMENT) { @@ -549,6 +558,8 @@ private static void parseSslContextType(final ConfigurationXMLStreamReader reade final ExceptionSupplier finalKeyManagerSupplier = keyManagerSupplier; final ExceptionSupplier finalTrustStoreSupplier = trustStoreSupplier; final boolean initTrustManager = finalTrustStoreSupplier != null || isSet(foundBits, 7); + final boolean finalAcceptOcspStapling = acceptOcspStapling; + sslContextsMap.putIfAbsent(name, () -> { final SSLContextBuilder sslContextBuilder = new SSLContextBuilder(); sslContextBuilder.setClientMode(true); @@ -574,6 +585,7 @@ private static void parseSslContextType(final ConfigurationXMLStreamReader reade sslContextBuilder.setProviderName(finalProviderName); sslContextBuilder.setProviderSupplier(finalProvidersSupplier); sslContextBuilder.setUseCipherSuitesOrder(true); + sslContextBuilder.setAcceptOCSPStapling(finalAcceptOcspStapling); return sslContextBuilder.build(); }); return; @@ -582,6 +594,56 @@ private static void parseSslContextType(final ConfigurationXMLStreamReader reade throw reader.unexpectedDocumentEnd(); } + private static boolean parseOcspStaplingType(ConfigurationXMLStreamReader reader, TrustManagerBuilder builder, Version xmlVersion, Map> keyStoresMap) throws ConfigXMLParseException { + final int attributeCount = reader.getAttributeCount(); + boolean acceptOcspStapling = false; + boolean softFail = false; + boolean gotResponderCertAlias = false; + boolean gotResponderKeystore = false; + + for (int i = 0; i < attributeCount; i ++) { + checkAttributeNamespace(reader, i); + switch (reader.getAttributeLocalName(i)) { + case "accept-ocsp": { + if (acceptOcspStapling) throw reader.unexpectedAttribute(i); + if (!xmlVersion.isAtLeast(Version.VERSION_1_8)) throw reader.unexpectedAttribute(i); + acceptOcspStapling = reader.getBooleanAttributeValueResolved(i); + builder.setOcspStapling(acceptOcspStapling); + break; + } + case "soft-fail": { + if (softFail) throw reader.unexpectedAttribute(i); + if (!xmlVersion.isAtLeast(Version.VERSION_1_8)) throw reader.unexpectedAttribute(i); + softFail = reader.getBooleanAttributeValueResolved(i); + builder.setSoftFail(softFail); + break; + } + case "responder-certificate": { + if (gotResponderCertAlias) throw reader.unexpectedAttribute(i); + builder.setOcspRescponderCertAlias(reader.getAttributeValueResolved(i)); + gotResponderCertAlias = true; + break; + } + case "responder-keystore": { + if (gotResponderKeystore) throw reader.unexpectedAttribute(i); + builder.setOcspResponderCertKeystoreSupplier(keyStoresMap.get(reader.getAttributeValueResolved(i))); + gotResponderKeystore = true; + break; + } + default: throw reader.unexpectedAttribute(i); + } + } + while (reader.hasNext()) { + final int tag = reader.nextTag(); + if (tag == END_ELEMENT) { + return acceptOcspStapling; + } else { + throw reader.unexpectedContent(); + } + } + throw reader.unexpectedDocumentEnd(); + } + private static class TrustManagerBuilder { final Supplier providers; final Location xmlLocation; @@ -592,6 +654,7 @@ private static class TrustManagerBuilder { List crlStreams = new ArrayList<>(); int maxCertPath = 5; boolean ocsp = false; + boolean ocspStapling = false; boolean preferCrls = false; boolean onlyLeafCert = false; boolean softFail = false; @@ -638,6 +701,9 @@ boolean isMaxCertPathSet() { public void setOcsp() { this.ocsp = true; } + public void setOcspStapling(boolean ocspStapling) { + this.ocspStapling = ocspStapling; + } public void setPreferCrls(boolean preferCrls) { this.preferCrls = preferCrls; @@ -697,6 +763,15 @@ X509TrustManager build() throws NoSuchAlgorithmException, KeyStoreException, Con revocationBuilder.setOcspResponderCert((X509Certificate) responderStore.getCertificate(responderCertAlias)); } + return revocationBuilder.build(); + } else if (ocspStapling) { + X509RevocationTrustManager.Builder revocationBuilder = X509RevocationTrustManager.builder(); + revocationBuilder.setTrustManagerFactory(trustManagerFactory); + revocationBuilder.setTrustStore(trustStore); + revocationBuilder.setCheckRevocation(true); + revocationBuilder.setSoftFail(softFail); + KeyStore responderStore = responderStoreSupplier != null ? responderStoreSupplier.get() : trustStore; + revocationBuilder.setOcspResponderCert((X509Certificate) responderStore.getCertificate(responderCertAlias)); return revocationBuilder.build(); } else { trustManagerFactory.init(trustStore); diff --git a/auth/client/src/main/resources/schema/elytron-client-1_8.xsd b/auth/client/src/main/resources/schema/elytron-client-1_8.xsd index 104fecbb268..ada98c61381 100644 --- a/auth/client/src/main/resources/schema/elytron-client-1_8.xsd +++ b/auth/client/src/main/resources/schema/elytron-client-1_8.xsd @@ -100,6 +100,13 @@ + + + + Indicated whether the client will accept OCSP stapled responses from the server. + + + @@ -121,6 +128,38 @@ + + + + + Indicates whether the client accepts ocsp stapled requests from the server. + + + + + + + Determines the client's behaviour upon receiving a certificate with unknown + revocation status from the server. + + + + + + + Alias of OCSP Responder certificate. + + + + + + + Keystore for OCSP Responder certificate. trust-manager keystore is used by default and responder-certificate has to be defined. + + + + + diff --git a/auth/client/src/test/java/org/wildfly/security/auth/client/ElytronXmlParserTest.java b/auth/client/src/test/java/org/wildfly/security/auth/client/ElytronXmlParserTest.java index ac33fd96f04..f7c259d9816 100644 --- a/auth/client/src/test/java/org/wildfly/security/auth/client/ElytronXmlParserTest.java +++ b/auth/client/src/test/java/org/wildfly/security/auth/client/ElytronXmlParserTest.java @@ -194,6 +194,14 @@ public void testCipherSuites() throws Exception { checkSSLContext(authContext, "http://names-only.org"); } + @Test + public void checkSSLContextWithOCSPStapling() throws Exception { + URL config = getClass().getResource("test-wildfly-config-v1_8.xml"); + SecurityFactory authContext = ElytronXmlParser.parseAuthenticationClientConfiguration(config.toURI()); + Assert.assertNotNull(authContext); + Assert.assertNotNull(authContext.create().getSslRules()); + } + private void checkSSLContext(SecurityFactory authContext, String uri) throws Exception { RuleNode> node = authContext.create().sslRuleMatching(new URI(uri), null, null); Assert.assertNotNull(node); diff --git a/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml b/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml new file mode 100644 index 00000000000..78abfa5715e --- /dev/null +++ b/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml @@ -0,0 +1,64 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/ssl/src/main/java/org/wildfly/security/ssl/ElytronMessages.java b/ssl/src/main/java/org/wildfly/security/ssl/ElytronMessages.java index 3f9efdbf2e7..6febc9d084e 100644 --- a/ssl/src/main/java/org/wildfly/security/ssl/ElytronMessages.java +++ b/ssl/src/main/java/org/wildfly/security/ssl/ElytronMessages.java @@ -193,4 +193,7 @@ interface ElytronMessages extends BasicLogger { @Message(id = 15001, value = "No '%s' provided by the configured providers") NoSuchAlgorithmException noSslContextProvided(String type); + + @Message(id = 15002, value = "ResponderURI needs to be provided to override AIA extension.") + IllegalArgumentException responderURIRequired(); } diff --git a/ssl/src/main/java/org/wildfly/security/ssl/SSLContextBuilder.java b/ssl/src/main/java/org/wildfly/security/ssl/SSLContextBuilder.java index 5ca5e0e24be..88dff502139 100644 --- a/ssl/src/main/java/org/wildfly/security/ssl/SSLContextBuilder.java +++ b/ssl/src/main/java/org/wildfly/security/ssl/SSLContextBuilder.java @@ -77,6 +77,13 @@ public final class SSLContextBuilder { private String providerName; private boolean wrap = true; private MechanismConfigurationSelector mechanismConfigurationSelector; + private int responseTimeout; + private int cacheSize; + private int cacheLifetime; + private String responderURI; + private boolean responderOverride; + private boolean ignoreExtensions; + private boolean acceptOCSPStapling; /** * Set the security domain to use to authenticate clients. @@ -309,7 +316,82 @@ public SSLContextBuilder setMechanismConfigurationSelector(final MechanismConfig return this; } + /** + * Set the response timeout, which will be used for OCSP Stapling. + * + * @param responseTimeout controls the maximum amount of time the server will use to obtain OCSP responses, whether from the cache or by contacting an OCSP responder. + * @return this builder + */ + public SSLContextBuilder setResponseTimeout(final int responseTimeout) { + this.responseTimeout = responseTimeout; + return this; + } + + /** + * Set the cache size, which will be used for OCSP Stapling. + * + * @param cacheSize controls the maximum cache size in entries. + * @return this builder + */ + public SSLContextBuilder setCacheSize(final int cacheSize) { + this.cacheSize = cacheSize; + return this; + } + + /** + * Set the cache lifetime, which will be used for OCSP Stapling. + * + * @param cacheLifetime controls the maximum life of a cached response. + * @return this builder + */ + public SSLContextBuilder setCacheLifetime(final int cacheLifetime) { + this.cacheLifetime = cacheLifetime; + return this; + } + + /** + * Set the responder URI, which will be used for OCSP Stapling. + * + * @param responderURI enables the administrator to set a default URI in the event that certificates used for TLS do not have the Authority Info Access (AIA) extension. + * @return this builder + */ + public SSLContextBuilder setResponderURI(final String responderURI) { + this.responderURI = responderURI; + return this; + } + + /** + * Set the responder override property, which will be used for OCSP Stapling. + * + * @param responderOverride enables a URI provided through the jdk.tls.stapling.responderURI property to override any AIA extension value. + * @return this builder + */ + public SSLContextBuilder setResponderOverride(final boolean responderOverride) { + this.responderOverride = responderOverride; + return this; + } + + /** + * Set the ignore extensions property, which will be used for OCSP Stapling. + * + * @param ignoreExtensions disables the forwarding of OCSP extensions specified in the status_request or status_request_v2 TLS extensions. + * @return this builder + */ + public SSLContextBuilder setIgnoreExtensions(final boolean ignoreExtensions) { + this.ignoreExtensions = ignoreExtensions; + return this; + } + /** + * Indicates whether the client will accept OCSP Stapled responses from the server. + * + * @param acceptOCSPStapling will enable or disable client side OCSP stapling. + * @return this builder + */ + public SSLContextBuilder setAcceptOCSPStapling(final boolean acceptOCSPStapling) { + this.acceptOCSPStapling = acceptOCSPStapling; + return this; + } /** * Build a security factory for the new context. The factory will cache the constructed instance. @@ -331,10 +413,40 @@ public SecurityFactory build() { final boolean needClientAuth = this.needClientAuth; final boolean useCipherSuitesOrder = this.useCipherSuitesOrder; final boolean wrap = this.wrap; + final int responseTimeout = this.responseTimeout; + final int cacheSize = this.cacheSize; + final String responderURI = this.responderURI; + final boolean responderOverride = this.responderOverride; + final boolean ignoreExtensions = this.ignoreExtensions; + final boolean acceptOCSPStapling = this.acceptOCSPStapling; final MechanismConfigurationSelector mechanismConfigurationSelector = this.mechanismConfigurationSelector != null ? this.mechanismConfigurationSelector : MechanismConfigurationSelector.constantSelector(MechanismConfiguration.EMPTY); + if (clientMode && acceptOCSPStapling) { // client-ssl-context + // enable client side OCSP fall back + Security.setProperty("ocsp.enable", Boolean.TRUE.toString()); + + // Enable client side support for accepting OCSP stapling + System.setProperty("jdk.tls.client.enableStatusRequestExtension", Boolean.TRUE.toString()); + + } else if (!clientMode && checkOCSPStaplingEnabled()) { //server-ssl-context + // Enable server side support for OCSP stapling + System.setProperty("jdk.tls.server.enableStatusRequestExtension", Boolean.TRUE.toString()); + + // Set additional properties related to the server side OCSP Stapling + System.setProperty("jdk.tls.stapling.responseTimeout", String.valueOf(responseTimeout)); + System.setProperty("jdk.tls.stapling.cacheSize", String.valueOf(cacheSize)); + System.setProperty("jdk.tls.stapling.cacheLifetime", String.valueOf(cacheLifetime)); + if (responderURI != null) + System.setProperty("jdk.tls.stapling.responderURI", responderURI); + if (responderOverride && responderURI.isEmpty()) { + throw ElytronMessages.log.responderURIRequired(); + } + System.setProperty("jdk.tls.stapling.responderOverride", String.valueOf(responderOverride)); + System.setProperty("jdk.tls.stapling.ignoreExtensions", String.valueOf(ignoreExtensions)); + } + return new OneTimeSecurityFactory<>(() -> { final SecurityFactory sslContextFactory = SSLUtils.createSslContextFactory(protocolSelector, providerSupplier, providerName); // construct the original context @@ -359,10 +471,18 @@ public SecurityFactory build() { " wantClientAuth = %s%n" + " needClientAuth = %s%n" + " useCipherSuitesOrder = %s%n" + - " wrap = %s%n", + " wrap = %s%n + " + + " acceptOCSPStapling = %s%n + " + + " responseTimeout = %s%n + " + + " cacheSize = %s%n + " + + " cacheLifetime = %s%n + " + + " responderURI = %s%n + " + + " responderOverride = %s%n + " + + " ignoreExtensions = %s%n:", securityDomain, canAuthPeers, cipherSuiteSelector, protocolSelector, x509TrustManager, x509KeyManager, providerSupplier, clientMode, authenticationOptional, sessionCacheSize, - sessionTimeout, wantClientAuth, needClientAuth, useCipherSuitesOrder, wrap); + sessionTimeout, wantClientAuth, needClientAuth, useCipherSuitesOrder, wrap, acceptOCSPStapling, + responseTimeout, cacheSize, cacheLifetime, responderURI, responderOverride, ignoreExtensions); } sslContext.init(x509KeyManager == null ? null : new KeyManager[]{ @@ -387,4 +507,8 @@ public SecurityFactory build() { return new DelegatingSSLContext(contextSpi); }); } + + private boolean checkOCSPStaplingEnabled() { + return (this.responseTimeout > 0) && (this.cacheSize != 0) && (this.cacheLifetime != 0); + } } diff --git a/ssl/src/main/java/org/wildfly/security/ssl/X509RevocationTrustManager.java b/ssl/src/main/java/org/wildfly/security/ssl/X509RevocationTrustManager.java index 8e14fa93089..d62f06650a2 100644 --- a/ssl/src/main/java/org/wildfly/security/ssl/X509RevocationTrustManager.java +++ b/ssl/src/main/java/org/wildfly/security/ssl/X509RevocationTrustManager.java @@ -73,44 +73,60 @@ private X509RevocationTrustManager(Builder builder) { try { PKIXBuilderParameters params = new PKIXBuilderParameters(builder.trustStore, new X509CertSelector()); - if (builder.crlStreams != null && ! builder.crlStreams.isEmpty()) { - CertStoreParameters csp = new CollectionCertStoreParameters(getCRLs(builder.crlStreams)); - CertStore store = CertStore.getInstance("Collection", csp); - params.addCertStore(store); - } + if (builder.checkRevocation) { // for ocspStapling + CertPathBuilder cpb = CertPathBuilder.getInstance("PKIX"); + PKIXRevocationChecker rc = (PKIXRevocationChecker) cpb.getRevocationChecker(); - CertPathBuilder cpb = CertPathBuilder.getInstance("PKIX"); - PKIXRevocationChecker rc = (PKIXRevocationChecker) cpb.getRevocationChecker(); + if (builder.softFail) + rc.setOptions(EnumSet.of(PKIXRevocationChecker.Option.SOFT_FAIL)); - if (builder.ocspResponderCert != null) { - rc.setOcspResponderCert(builder.ocspResponderCert); - } + if (builder.ocspResponderCert != null) { + rc.setOcspResponderCert(builder.ocspResponderCert); + } - EnumSet options = EnumSet.noneOf(PKIXRevocationChecker.Option.class); - if (builder.onlyEndEntity) { - options.add(PKIXRevocationChecker.Option.ONLY_END_ENTITY); - } - if (builder.preferCrls) { - options.add(PKIXRevocationChecker.Option.PREFER_CRLS); - } - if (builder.softFail) { - options.add(PKIXRevocationChecker.Option.SOFT_FAIL); - } - if (builder.noFallback) { - options.add(PKIXRevocationChecker.Option.NO_FALLBACK); - } + params.addCertPathChecker(rc); + } else { // for other revocation checking - rc.setOptions(options); - rc.setOcspResponder(builder.responderUri); - params.setRevocationEnabled(true); - params.addCertPathChecker(rc); + if (builder.crlStreams != null && ! builder.crlStreams.isEmpty()) { + CertStoreParameters csp = new CollectionCertStoreParameters(getCRLs(builder.crlStreams)); + CertStore store = CertStore.getInstance("Collection", csp); + params.addCertStore(store); + } - PKIXCertPathChecker maxPathLengthChecker = new MaxPathLengthChecker(builder.maxCertPath); - params.addCertPathChecker(maxPathLengthChecker); - params.setMaxPathLength(builder.maxCertPath); + CertPathBuilder cpb = CertPathBuilder.getInstance("PKIX"); + PKIXRevocationChecker rc = (PKIXRevocationChecker) cpb.getRevocationChecker(); - builder.trustManagerFactory.init(new CertPathTrustManagerParameters(params)); + if (builder.ocspResponderCert != null) { + rc.setOcspResponderCert(builder.ocspResponderCert); + } + EnumSet options = EnumSet.noneOf(PKIXRevocationChecker.Option.class); + if (builder.onlyEndEntity) { + options.add(PKIXRevocationChecker.Option.ONLY_END_ENTITY); + } + if (builder.preferCrls) { + options.add(PKIXRevocationChecker.Option.PREFER_CRLS); + } + if (builder.softFail) { + options.add(PKIXRevocationChecker.Option.SOFT_FAIL); + } + if (builder.noFallback) { + options.add(PKIXRevocationChecker.Option.NO_FALLBACK); + } + + rc.setOptions(options); + if (builder.responderUri != null) { + rc.setOcspResponder(builder.responderUri); + } + params.setRevocationEnabled(true); + params.addCertPathChecker(rc); + + PKIXCertPathChecker maxPathLengthChecker = new MaxPathLengthChecker(builder.maxCertPath); + params.addCertPathChecker(maxPathLengthChecker); + params.setMaxPathLength(builder.maxCertPath); + + } + builder.trustManagerFactory.init(new CertPathTrustManagerParameters(params)); X509TrustManager[] trustManagers = Stream.of(builder.trustManagerFactory.getTrustManagers()).map(trustManager -> trustManager instanceof X509TrustManager ? (X509TrustManager) trustManager : null).filter(Objects::nonNull).toArray(X509TrustManager[]::new); if (trustManagers.length == 0) { @@ -195,6 +211,7 @@ public static class Builder { private boolean onlyEndEntity = false; private boolean softFail = false; private boolean noFallback = false; + private boolean checkRevocation = false; private Builder() {} @@ -325,6 +342,17 @@ public Builder setNoFallback(boolean noFallback) { return this; } + /** + * Enables revocation checking on the client side. Default false + * + * @param checkRevocation determines whether revocation checking should be enabled on the client side + * @return this Builder for subsequent changes + */ + public Builder setCheckRevocation(boolean checkRevocation) { + this.checkRevocation = checkRevocation; + return this; + } + /** * Set OCSP responder's certificate. By default issuer certificate of certificate being validated is used. * diff --git a/tests/base/src/test/java/org/wildfly/security/ssl/SSLAuthenticationTest.java b/tests/base/src/test/java/org/wildfly/security/ssl/SSLAuthenticationTest.java index b1005514045..dd3fa4c00d4 100644 --- a/tests/base/src/test/java/org/wildfly/security/ssl/SSLAuthenticationTest.java +++ b/tests/base/src/test/java/org/wildfly/security/ssl/SSLAuthenticationTest.java @@ -104,12 +104,12 @@ public class SSLAuthenticationTest { private final int TESTING_PORT = 18201; private static final char[] PASSWORD = "Elytron".toCharArray(); - private static final String JKS_LOCATION = "./target/test-classes/pkcs12"; + private static final String PKCS_LOCATION = "./target/test-classes/pkcs12"; private static final String CA_CRL_LOCATION = "./target/test-classes/ca/crl"; private static final String ICA_CRL_LOCATION = "./target/test-classes/ica/crl"; private static final File WORKING_DIR_CACRL = new File(CA_CRL_LOCATION); private static final File WORKING_DIR_ICACRL = new File(ICA_CRL_LOCATION); - private static final File SHORTWINGED_FILE = new File(JKS_LOCATION, "shortwinged.keystore"); + private static final File SHORTWINGED_FILE = new File(PKCS_LOCATION, "shortwinged.keystore"); private static final File CA_BLANK_PEM_CRL = new File(WORKING_DIR_CACRL, "blank.pem"); private static final File ICA_BLANK_PEM_CRL = new File(WORKING_DIR_ICACRL, "blank.pem"); private static final File BLANK_BLANK_PEM_CRL = new File(WORKING_DIR_ICACRL, "blank-blank.pem"); @@ -168,7 +168,7 @@ public static void beforeTest() throws Exception { WORKING_DIR_ICACRL.mkdirs(); caGenerationTool = CAGenerationTool.builder() - .setBaseDir(JKS_LOCATION) + .setBaseDir(PKCS_LOCATION) .setRequestIdentities(Identity.values()) // Create all identities. .build(); @@ -182,6 +182,7 @@ public static void beforeTest() throws Exception { "ocsp-responder.keystore", new ExtendedKeyUsageExtension(false, Collections.singletonList(OID_KP_OCSP_SIGNING))); ocspResponderCertificate = responderIdentity.getCertificate(); + /* =================== CLIENT SIDE OCSP RELATED CERTS =================== */ // Generates GOOD certificate referencing the OCSP responder goodIdentity = intermediateCAIdentity.createIdentity("checked", new X500Principal("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedGood"), @@ -209,14 +210,39 @@ public static void beforeTest() throws Exception { X509Certificate greenJuneCertificate = caGenerationTool .getDefinedIdentity(Identity.GREENJUNE) .getCertificate(); + /* =================== OCSP STAPLING RELATED CERTS =================== */ + // Generates GOOD certificate referencing the OCSP responder + X509Certificate ocspCheckedGoodServerCertificate = caGenerationTool.createIdentity("checked", + new X500Principal("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerGood"), + "ocsp-stapling-checked-good.keystore", Identity.CA, new AuthorityInformationAccessExtension(Collections.singletonList( + new AccessDescription(OID_AD_OCSP, new GeneralName.URIName("http://localhost:" + OCSP_PORT + "/ocsp")) + ))); + + // Generates REVOKED certificate referencing the OCSP responder + X509Certificate ocspCheckedRevokedServerCertificate = caGenerationTool.createIdentity("checked", + new X500Principal("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerRevoked"), + "ocsp-stapling-checked-revoked.keystore", Identity.CA, (new AuthorityInformationAccessExtension(Collections.singletonList( + new AccessDescription(OID_AD_OCSP, new GeneralName.URIName("http://localhost:" + OCSP_PORT + "/ocsp")) + )))); + + // Generates UNKNOWN certificate referencing the OCSP responder + X509Certificate ocspCheckedUnknownServerCertificate = caGenerationTool.createIdentity("checked", + new X500Principal("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerUnknown"), + "ocsp-stapling-checked-unknown.keystore", Identity.CA, new AuthorityInformationAccessExtension(Collections.singletonList( + new AccessDescription(OID_AD_OCSP, new GeneralName.URIName("http://localhost:" + OCSP_PORT + "/ocsp")) + ))); + KeyStore beetlesKeyStore = caGenerationTool.getBeetlesKeyStore(); beetlesKeyStore.setCertificateEntry("ocspResponder", ocspResponderCertificate); beetlesKeyStore.setCertificateEntry("ocspCheckedGood", ocspCheckedGoodCertificate); + beetlesKeyStore.setCertificateEntry("ocspCheckedServerGood", ocspCheckedGoodServerCertificate); beetlesKeyStore.setCertificateEntry("ocspCheckedRevoked", ocspCheckedRevokedCertificate); + beetlesKeyStore.setCertificateEntry("ocspCheckedServerRevoked", ocspCheckedRevokedServerCertificate); beetlesKeyStore.setCertificateEntry("ocspCheckedUnknown", ocspCheckedUnknownCertificate); + beetlesKeyStore.setCertificateEntry("ocspCheckedServerUnknown", ocspCheckedUnknownServerCertificate); beetlesKeyStore.setCertificateEntry("green june", greenJuneCertificate); - createTemporaryKeyStoreFile(beetlesKeyStore, new File(JKS_LOCATION, "beetles.keystore"), PASSWORD); + createTemporaryKeyStoreFile(beetlesKeyStore, new File(PKCS_LOCATION, "beetles.keystore"), PASSWORD); // Adds trusted cert for shortwinged shortWingedKeyStore = createKeyStore(); @@ -352,7 +378,10 @@ public static void beforeTest() throws Exception { ocspServer.createCertificate(1, 1, intermediateCAIdentity.getCertificate()); ocspServer.createCertificate(2, 2, ocspCheckedGoodCertificate); ocspServer.createCertificate(3, 1, ocspCheckedRevokedCertificate); + ocspServer.createCertificate(4, 1, ocspCheckedGoodServerCertificate); + ocspServer.createCertificate(5, 1, ocspCheckedRevokedServerCertificate); ocspServer.revokeCertificate(3, 4); + ocspServer.revokeCertificate(5, 4); ocspServer.start(); } @@ -823,6 +852,94 @@ public void testClientSideOcsp() throws Throwable { } } + @Test + public void testOcspStaplingGood() throws Throwable { + DefinedCAIdentity ca = caGenerationTool.getDefinedCAIdentity(Identity.CA); + DefinedIdentity ocspStaplingGood = caGenerationTool.getDefinedIdentity(Identity.OCSP_STAPLING_GOOD); + SSLContext serverContext = new SSLContextBuilder() + .setSecurityDomain(getKeyStoreBackedSecurityDomain(caGenerationTool.getBeetlesKeyStore())) + .setKeyManager(ocspStaplingGood.createKeyManager()) + .setTrustManager(ca.createTrustManager()) + .setNeedClientAuth(true) + .setResponseTimeout(5000) + .setCacheSize(256) + .setCacheLifetime(3600) + .build().create(); + + performConnectionTest(serverContext, "protocol://test-two-way-ocsp-stapling-good.org", true, "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=ocspCheckedServerGood", + "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=Ladybird", false); + } + + @Test + public void testOcspStaplingRevoked() throws Throwable { + DefinedCAIdentity ca = caGenerationTool.getDefinedCAIdentity(Identity.CA); + DefinedIdentity ocspStaplingRevoked = caGenerationTool.getDefinedIdentity(Identity.OCSP_STAPLING_REVOKED); + SSLContext serverContext = new SSLContextBuilder() + .setSecurityDomain(getKeyStoreBackedSecurityDomain(caGenerationTool.getBeetlesKeyStore())) + .setKeyManager(ocspStaplingRevoked.createKeyManager()) + .setTrustManager(ca.createTrustManager()) + .setNeedClientAuth(true) + .setResponseTimeout(5000) + .setCacheSize(256) + .setCacheLifetime(3600) + .build().create(); + + performConnectionTest(serverContext, "protocol://test-two-way-ocsp-stapling-revoked.org", false, "OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerRevoked", + "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=Ladybird", false); + } + + @Test + public void testOcspStaplingUnknownHardFail() throws Throwable { + DefinedCAIdentity ca = caGenerationTool.getDefinedCAIdentity(Identity.CA); + DefinedIdentity ocspStaplingUnknown = caGenerationTool.getDefinedIdentity(Identity.OCSP_STAPLING_UNKNOWN); + SSLContext serverContext = new SSLContextBuilder() + .setSecurityDomain(getKeyStoreBackedSecurityDomain(caGenerationTool.getBeetlesKeyStore())) + .setKeyManager(ocspStaplingUnknown.createKeyManager()) + .setTrustManager(ca.createTrustManager()) + .setNeedClientAuth(true) + .setResponseTimeout(5000) + .setCacheSize(256) + .setCacheLifetime(3600) + .build().create(); + + performConnectionTest(serverContext, "protocol://test-two-way-ocsp-stapling-unknown.org", false, "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=ocspCheckedServerUnknown", + "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=Ladybird", false); + } + + @Test + public void testOcspStaplingUnknownSoftFail() throws Throwable { + DefinedCAIdentity ca = caGenerationTool.getDefinedCAIdentity(Identity.CA); + DefinedIdentity ocspStaplingUnknown = caGenerationTool.getDefinedIdentity(Identity.OCSP_STAPLING_UNKNOWN); + SSLContext serverContext = new SSLContextBuilder() + .setSecurityDomain(getKeyStoreBackedSecurityDomain(caGenerationTool.getBeetlesKeyStore())) + .setKeyManager(ocspStaplingUnknown.createKeyManager()) + .setTrustManager(ca.createTrustManager()) + .setNeedClientAuth(true) + .setResponseTimeout(5000) + .setCacheSize(256) + .setCacheLifetime(3600) + .build().create(); + + performConnectionTest(serverContext, "protocol://test-two-way-ocsp-stapling-unknown-soft-fail.org", true, "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=ocspCheckedServerUnknown", + "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=Ladybird", false); + } + + @Test + public void testOcspStaplingOneWayGood() throws Throwable { + DefinedCAIdentity ca = caGenerationTool.getDefinedCAIdentity(Identity.CA); + DefinedIdentity ocspStaplingGood = caGenerationTool.getDefinedIdentity(Identity.OCSP_STAPLING_GOOD); + SSLContext serverContext = new SSLContextBuilder() + .setSecurityDomain(getKeyStoreBackedSecurityDomain(caGenerationTool.getBeetlesKeyStore())) + .setKeyManager(ocspStaplingGood.createKeyManager()) + .setResponseTimeout(5000) + .setCacheSize(256) + .setCacheLifetime(3600) + .build().create(); + + performConnectionTest(serverContext, "protocol://test-one-way-ocsp-stapling-good.org", true, "OU=Elytron,O=Elytron,C=UK,ST=Elytron,CN=ocspCheckedServerGood", + null, true); + } + @Test public void testWantClientAuthWithCorrectCertificate() throws Throwable { DefinedCAIdentity ca = caGenerationTool.getDefinedCAIdentity(Identity.CA); @@ -886,56 +1003,56 @@ private void testCommunication(SSLContext serverContext, SSLContext clientContex SSLSocket clientSocket = (SSLSocket) clientContext.getSocketFactory().createSocket("localhost", TESTING_PORT); SSLSocket serverSocket = (SSLSocket) listeningSocket.accept(); - ExecutorService serverExecutorService = Executors.newSingleThreadExecutor(); - Future serverFuture = serverExecutorService.submit(() -> { + ExecutorService clientExecutorService = Executors.newSingleThreadExecutor(); + Future clientFuture = clientExecutorService.submit(() -> { try { byte[] received = new byte[2]; - serverSocket.getInputStream().read(received); - serverSocket.getOutputStream().write(new byte[]{0x56, 0x78}); + clientSocket.getOutputStream().write(new byte[]{0x12, 0x34}); + clientSocket.getInputStream().read(received); - if (expectedClientPrincipal != null) { - assertEquals(expectedClientPrincipal, serverSocket.getSession().getPeerPrincipal().getName()); + if (expectedServerPrincipal != null) { + assertEquals(expectedServerPrincipal, clientSocket.getSession().getPeerPrincipal().getName()); } - SecurityIdentity identity = (SecurityIdentity) serverSocket.getSession().getValue(SSLUtils.SSL_SESSION_IDENTITY_KEY); if (oneWay) { - assertNull(identity); + assertNotEquals("TLSv1.3", clientSocket.getSession().getProtocol());// since TLS 1.3 is not enabled by default (ELY-1917) } else { - assertNotNull(identity); + assertNotEquals("TLSv1.3", serverSocket.getSession().getProtocol()); // since TLS 1.3 is not enabled by default + assertNotEquals("TLSv1.3", clientSocket.getSession().getProtocol()); // since TLS 1.3 is not enabled by default } - return received; } catch (Exception e) { - throw new RuntimeException("Server exception", e); + throw new RuntimeException("Client exception", e); } }); - ExecutorService clientExecutorService = Executors.newSingleThreadExecutor(); - Future clientFuture = clientExecutorService.submit(() -> { + ExecutorService serverExecutorService = Executors.newSingleThreadExecutor(); + Future serverFuture = serverExecutorService.submit(() -> { try { byte[] received = new byte[2]; - clientSocket.getOutputStream().write(new byte[]{0x12, 0x34}); - clientSocket.getInputStream().read(received); + serverSocket.getInputStream().read(received); + serverSocket.getOutputStream().write(new byte[]{0x56, 0x78}); - if (expectedServerPrincipal != null) { - assertEquals(expectedServerPrincipal, clientSocket.getSession().getPeerPrincipal().getName()); + if (expectedClientPrincipal != null) { + assertEquals(expectedClientPrincipal, serverSocket.getSession().getPeerPrincipal().getName()); } + SecurityIdentity identity = (SecurityIdentity) serverSocket.getSession().getValue(SSLUtils.SSL_SESSION_IDENTITY_KEY); if (oneWay) { - assertNotEquals("TLSv1.3", clientSocket.getSession().getProtocol());// since TLS 1.3 is not enabled by default (ELY-1917) + assertNull(identity); } else { - assertNotEquals("TLSv1.3", serverSocket.getSession().getProtocol()); // since TLS 1.3 is not enabled by default - assertNotEquals("TLSv1.3", clientSocket.getSession().getProtocol()); // since TLS 1.3 is not enabled by default + assertNotNull(identity); } + return received; } catch (Exception e) { - throw new RuntimeException("Client exception", e); + throw new RuntimeException("Server exception", e); } }); try { - assertArrayEquals(new byte[]{0x12, 0x34}, serverFuture.get()); assertArrayEquals(new byte[]{0x56, 0x78}, clientFuture.get()); + assertArrayEquals(new byte[]{0x12, 0x34}, serverFuture.get()); } catch (ExecutionException e) { if (e.getCause() != null && e.getCause() instanceof RuntimeException && e.getCause().getCause() != null) { throw e.getCause().getCause(); // unpack diff --git a/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml b/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml index 2f5ab506492..e9a666a51db 100644 --- a/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml +++ b/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml @@ -20,7 +20,7 @@ - + @@ -62,6 +62,22 @@ + + + + + + + + + + + + + + + + @@ -206,6 +222,53 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -256,6 +319,21 @@ + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/tests/common/src/test/java/org/wildfly/security/ssl/test/util/CAGenerationTool.java b/tests/common/src/test/java/org/wildfly/security/ssl/test/util/CAGenerationTool.java index e28f884d999..f0d81838e6b 100644 --- a/tests/common/src/test/java/org/wildfly/security/ssl/test/util/CAGenerationTool.java +++ b/tests/common/src/test/java/org/wildfly/security/ssl/test/util/CAGenerationTool.java @@ -431,7 +431,13 @@ public enum Identity { LADYBUG("OU=Wildfly, O=Wildfly, C=CA, ST=Wildfly, CN=Ladybug", SECOND_CA, false, "ladybug.keystore"), GREENJUNE("OU=Wildfly, O=Wildfly, C=CA, ST=Wildfly, CN=Green June", SECOND_CA, false, - "greenjune.keystore"); + "greenjune.keystore"), + OCSP_STAPLING_GOOD("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerGood", CA, false, + "ocsp-stapling-checked-good.keystore"), + OCSP_STAPLING_REVOKED("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerRevoked", CA, false, + "ocsp-stapling-checked-revoked.keystore"), + OCSP_STAPLING_UNKNOWN("OU=Elytron, O=Elytron, C=UK, ST=Elytron, CN=ocspCheckedServerUnknown", CA, false, + "ocsp-stapling-checked-unknown.keystore"); private final X500Principal principal; private final Identity signedBy; From 05fc2b9fa8399dc6732671824dd019b338235416 Mon Sep 17 00:00:00 2001 From: Prarthona Paul Date: Tue, 20 Aug 2024 09:42:43 -0400 Subject: [PATCH 3/3] Renamed the schema to include the stability level in it --- .../wildfly/security/auth/client/ElytronXmlParser.java | 8 ++++---- ...tron-client-1_8.xsd => elytron-client_preview_1_8.xsd} | 4 ++-- .../security/auth/client/test-wildfly-config-v1_8.xml | 2 +- .../wildfly/security/ssl/ssl-authentication-config.xml | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) rename auth/client/src/main/resources/schema/{elytron-client-1_8.xsd => elytron-client_preview_1_8.xsd} (99%) diff --git a/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java b/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java index 039344daf48..853b26faba0 100644 --- a/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java +++ b/auth/client/src/main/java/org/wildfly/security/auth/client/ElytronXmlParser.java @@ -155,7 +155,7 @@ private enum Version { VERSION_1_5("urn:elytron:client:1.5", VERSION_1_4), VERSION_1_6("urn:elytron:client:1.6", VERSION_1_5), VERSION_1_7("urn:elytron:client:1.7", VERSION_1_6), - VERSION_1_8("urn:elytron:client:1.8", VERSION_1_7); + VERSION_1_8_PREVIEW("urn:elytron:client:preview:1.8", VERSION_1_7); final String namespace; @@ -541,7 +541,7 @@ private static void parseSslContextType(final ConfigurationXMLStreamReader reade case "accept-ocsp-stapling": { if (isSet(foundBits, 10)) throw reader.unexpectedElement(); foundBits = setBit(foundBits, 10); - if (!xmlVersion.isAtLeast(Version.VERSION_1_8)) throw reader.unexpectedElement(); + if (!xmlVersion.isAtLeast(Version.VERSION_1_8_PREVIEW)) throw reader.unexpectedElement(); acceptOcspStapling = parseOcspStaplingType(reader, trustManagerBuilder, xmlVersion, keyStoresMap); break; } @@ -606,14 +606,14 @@ private static boolean parseOcspStaplingType(ConfigurationXMLStreamReader reader switch (reader.getAttributeLocalName(i)) { case "accept-ocsp": { if (acceptOcspStapling) throw reader.unexpectedAttribute(i); - if (!xmlVersion.isAtLeast(Version.VERSION_1_8)) throw reader.unexpectedAttribute(i); + if (!xmlVersion.isAtLeast(Version.VERSION_1_8_PREVIEW)) throw reader.unexpectedAttribute(i); acceptOcspStapling = reader.getBooleanAttributeValueResolved(i); builder.setOcspStapling(acceptOcspStapling); break; } case "soft-fail": { if (softFail) throw reader.unexpectedAttribute(i); - if (!xmlVersion.isAtLeast(Version.VERSION_1_8)) throw reader.unexpectedAttribute(i); + if (!xmlVersion.isAtLeast(Version.VERSION_1_8_PREVIEW)) throw reader.unexpectedAttribute(i); softFail = reader.getBooleanAttributeValueResolved(i); builder.setSoftFail(softFail); break; diff --git a/auth/client/src/main/resources/schema/elytron-client-1_8.xsd b/auth/client/src/main/resources/schema/elytron-client_preview_1_8.xsd similarity index 99% rename from auth/client/src/main/resources/schema/elytron-client-1_8.xsd rename to auth/client/src/main/resources/schema/elytron-client_preview_1_8.xsd index ada98c61381..5a982751fbc 100644 --- a/auth/client/src/main/resources/schema/elytron-client-1_8.xsd +++ b/auth/client/src/main/resources/schema/elytron-client_preview_1_8.xsd @@ -19,8 +19,8 @@ --> diff --git a/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml b/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml index 78abfa5715e..75d6b52fea5 100644 --- a/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml +++ b/auth/client/src/test/resources/org/wildfly/security/auth/client/test-wildfly-config-v1_8.xml @@ -15,7 +15,7 @@ limitations under the License. --> - + diff --git a/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml b/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml index e9a666a51db..963abd6a9be 100644 --- a/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml +++ b/tests/base/src/test/resources/org/wildfly/security/ssl/ssl-authentication-config.xml @@ -20,7 +20,7 @@ - +