Skip to content

Commit

Permalink
review comments addressed
Browse files Browse the repository at this point in the history
  • Loading branch information
trentjeff committed Sep 26, 2023
1 parent d2dac6d commit 7e706f2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@ protected ConfiguredTlsManager(String name,
*
* @return manager name
*/
// @Override // NamedService
public String name() {
return name;
}
Expand All @@ -84,7 +83,6 @@ public String name() {
*
* @return manager type
*/
// @Override // NamedService
public String type() {
return type;
}
Expand Down Expand Up @@ -129,7 +127,7 @@ public void init(WebServerTls tlsConfig) {
* @param tlsConfig the tls config
* @param keyManagers the key managers
* @param trustManagers the trust managers
* @deprecated this method will removed in a future release.
* @deprecated this method will be removed in a future release.
*/
@Deprecated
protected void initSslContext(WebServerTls tlsConfig,
Expand All @@ -152,7 +150,7 @@ protected void initSslContext(WebServerTls tlsConfig,
* @param tlsConfig the tls config
* @param keyManagers the key managers
* @param trustManagers the trust managers
* @deprecated this method will removed in a future release.
* @deprecated this method will be removed in a future release.
*/
@Deprecated
protected void reload(WebServerTls tlsConfig,
Expand Down Expand Up @@ -188,7 +186,7 @@ protected TrustManagerFactory trustAllTmf() {
*
* @param tlsConfig TLS config
* @return a new trust manager factory
* @deprecated this method will removed in a future release.
* @deprecated this method will be removed in a future release.
*/
@Deprecated
protected TrustManagerFactory createTmf(WebServerTls tlsConfig) {
Expand All @@ -204,7 +202,7 @@ protected TrustManagerFactory createTmf(WebServerTls tlsConfig) {
*
* @param tlsConfig TLS config
* @param sslContext ssl context to store
* @deprecated this method will removed in a future release.
* @deprecated this method will be removed in a future release.
*/
@Deprecated
protected void configureAndSet(WebServerTls tlsConfig,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ public interface TlsManager {
* Callers can subscribe to updates to be notified when the SSL context changes.
*
* @param sslContextConsumer the consumer that will receive the new/update context after it has been reloaded by the manager
* @deprecated this method will removed in a future release.
* @deprecated this method will be removed in a future release.
*/
@Deprecated
void subscribe(Consumer<SSLContext> sslContextConsumer);
Expand Down

0 comments on commit 7e706f2

Please sign in to comment.