handle of SNI has a problem

[zb564739784]

Questions

serverNameMapping method of io.vertx.core.net.impl.SslChannelProvider Class：

  public AsyncMapping<? super String, ? extends SslContext> serverNameMapping() {
    return (AsyncMapping<String, SslContext>) (serverName, promise) -> {
      workerPool.execute(() -> {
              ···
          int idx = idx(useAlpn);
          SslContext sslContext = sslContextMaps[idx].computeIfAbsent(serverName, s -> sslContextProvider.createServerContext(kmf, trustManagers, s, useAlpn));
          promise.setSuccess(sslContext);
        }
      });
      return promise;
    };

sslContextMaps[idx] add no restrictions based on "serverName"，If the client keeps making fake "serverName" it will cause memory leaks in sslContextMaps[idx]

Version

<vertx.version>4.5.1</vertx.version>

[vietj]

Thanks, this has been fixed by #5100 that will be released in Vert.x 4.5.3