You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2020/04/13 20:16:19 UTC

[logging-log4j2] 01/02: Fix verifyHostName SSL option

This is an automated email from the ASF dual-hosted git repository.

mattsicker pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git

commit ab1277088a447275e5bb8032af39ef0549be11ad
Author: Matt Sicker <bo...@gmail.com>
AuthorDate: Mon Apr 13 15:15:55 2020 -0500

    Fix verifyHostName SSL option
    
    This should be an attribute as there are no plugins corresponding to booleans.
    
    Signed-off-by: Matt Sicker <bo...@gmail.com>
---
 .../java/org/apache/logging/log4j/core/net/ssl/SslConfiguration.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/net/ssl/SslConfiguration.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/net/ssl/SslConfiguration.java
index bc89fbe..ad03552 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/net/ssl/SslConfiguration.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/net/ssl/SslConfiguration.java
@@ -251,7 +251,7 @@ public class SslConfiguration {
         @PluginAttribute("protocol") final String protocol,
         @PluginElement("KeyStore") final KeyStoreConfiguration keyStoreConfig,
         @PluginElement("TrustStore") final TrustStoreConfiguration trustStoreConfig,
-        @PluginElement("verifyHostName") final boolean verifyHostName) {
+        @PluginAttribute("verifyHostName") final boolean verifyHostName) {
         // @formatter:on
         return new SslConfiguration(protocol, keyStoreConfig, trustStoreConfig, verifyHostName);
     }