Github user NicoK commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6355#discussion_r203617345
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/configuration/SecurityOptions.java ---
    @@ -160,4 +160,41 @@
                key("security.ssl.verify-hostname")
                        .defaultValue(true)
                        .withDescription("Flag to enable peer’s hostname 
verification during ssl handshake.");
    +
    +   /**
    +    * SSL session cache size.
    +    */
    +   public static final ConfigOption<Integer> SSL_SESSION_CACHE_SIZE =
    +           key("security.ssl.session-cache-size")
    +                   .defaultValue(-1)
    +                   .withDescription("The size of the cache used for 
storing SSL session objects. "
    +                           + "According to 
https://github.com/netty/netty/issues/832, you should always set "
    +                           + "this to an appropriate number to not run 
into a bug with stalling IO threads "
    +                           + "during garbage collection. (-1 = use system 
default).");
    +
    +   /**
    +    * SSL session timeout.
    +    */
    +   public static final ConfigOption<Integer> SSL_SESSION_TIMEOUT =
    +           key("security.ssl.session-timeout")
    +                   .defaultValue(-1)
    +                   .withDescription("The timeout (in ms) for the cached 
SSL session objects. (-1 = use system default)");
    +
    +   /**
    +    * SSL session timeout during handshakes.
    +    */
    +   public static final ConfigOption<Integer> SSL_HANDSHAKE_TIMEOUT =
    +           key("security.ssl.handshake-timeout")
    +                   .defaultValue(-1)
    +                   .withDescription("The timeout (in ms) during SSL 
handshake. (-1 = use system default)");
    +
    +   /**
    +    * SSL session timeout after flushing the `close_notify` message.
    +    */
    +   public static final ConfigOption<Integer> 
SSL_CLOSE_NOTIFY_FLUSH_TIMEOUT =
    +           key("security.ssl.close-notify-flush-timeout")
    +                   .defaultValue(-1)
    +                   .withDescription("The timeout (in ms) for flushing the 
`close_notify` that was triggered by closing a " +
    --- End diff --
    
    unfortunately yes
    
    FYI: I found the difference:
    `The timeout (in ms) for flushing the close_notify that was triggered by 
closing a channel. If the close_notify was not flushed in the given timeout the 
channel will be closed  forcibly. (-1 = use system default)` vs. 
    `The timeout (in ms) for flushing the close_notify that was triggered by 
closing a channel. If the close_notify was not flushed in the given timeout the 
channel will be closed forcibly. (-1 = use system default)`
    -> seems like a double-space is made a single space at some 
point...fixing...


---

Reply via email to