You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2026/06/08 12:25:28 UTC
(tomcat) branch 10.1.x updated: Align OpenSSL/Panama TLS implementation with other implementations
This is an automated email from the ASF dual-hosted git repository.
markt-asf pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new feec60d609 Align OpenSSL/Panama TLS implementation with other implementations
feec60d609 is described below
commit feec60d6099727db6f911534f6a0f6926ebab070
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Mon Jun 8 13:24:40 2026 +0100
Align OpenSSL/Panama TLS implementation with other implementations
Throw an exception if there is an error loading the provided CRL(s).
---
java/org/apache/tomcat/util/net/openssl/panama/OpenSSLContext.java | 6 ++++--
webapps/docs/changelog.xml | 5 +++++
2 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/java/org/apache/tomcat/util/net/openssl/panama/OpenSSLContext.java b/java/org/apache/tomcat/util/net/openssl/panama/OpenSSLContext.java
index be103586b6..349b6997d9 100644
--- a/java/org/apache/tomcat/util/net/openssl/panama/OpenSSLContext.java
+++ b/java/org/apache/tomcat/util/net/openssl/panama/OpenSSLContext.java
@@ -1198,7 +1198,8 @@ public class OpenSSLContext implements org.apache.tomcat.util.net.SSLContext {
SSLHostConfig.adjustRelativePath(sslHostConfig.getCertificateRevocationListFile()));
if (X509_LOOKUP_load_file(x509Lookup, certificateRevocationListFileNative,
X509_FILETYPE_PEM()) <= 0) {
- log.error(sm.getString("openssl.errorLoadingCertificateRevocationListWithError",
+ throw new IllegalArgumentException(sm.getString(
+ "openssl.errorLoadingCertificateRevocationListWithError",
sslHostConfig.getCertificateRevocationListFile(), OpenSSLLibrary.getLastError()));
}
}
@@ -1208,7 +1209,8 @@ public class OpenSSLContext implements org.apache.tomcat.util.net.SSLContext {
SSLHostConfig.adjustRelativePath(sslHostConfig.getCertificateRevocationListPath()));
if (X509_LOOKUP_add_dir(x509Lookup, certificateRevocationListPathNative,
X509_FILETYPE_PEM()) <= 0) {
- log.error(sm.getString("openssl.errorLoadingCertificateRevocationListWithError",
+ throw new IllegalArgumentException(sm.getString(
+ "openssl.errorLoadingCertificateRevocationListWithError",
sslHostConfig.getCertificateRevocationListPath(), OpenSSLLibrary.getLastError()));
}
}
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 98e8664885..f12c72a2c4 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -301,6 +301,11 @@
<code>isTrailerFieldsReady</code> was always returning
<code>true</code>. (remm)
</fix>
+ <fix>
+ Align OpenSSL/Panama TLS implementation with other implementations and
+ throw an exception if there is an error loading the provided CRL(s).
+ (markt)
+ </fix>
</changelog>
</subsection>
<subsection name="Jasper">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org