Address review comments.

Author: kannanjgithub

netty/src/main/java/io/grpc/netty/NettyClientTransport.java

@@ -108,7 +108,6 @@ class NettyClientTransport implements ConnectionClientTransport {
   private final ChannelLogger channelLogger;
   private final boolean useGetForSafeMethods;
   private final Ticker ticker;
-  private final Logger logger = Logger.getLogger(NettyClientTransport.class.getName());
 
   NettyClientTransport(
       SocketAddress address,
@@ -204,9 +203,6 @@ public ClientStream newStream(
         if (enablePerRpcAuthorityCheck) {
           return new FailingClientStream(verificationStatus, tracers);
         }
-        logger.warning(String.format("Authority verification for the rpc %s failed (this will be an" +
-                " error in the future) with error status: %s", method.getFullMethodName(),
-                verificationStatus.getDescription()));
       }
     }
     StatsTraceContext statsTraceCtx =

netty/src/main/java/io/grpc/netty/ProtocolNegotiators.java

@@ -116,13 +116,12 @@ final class ProtocolNegotiators {
       EnumSet.of(
           TlsServerCredentials.Feature.MTLS, TlsServerCredentials.Feature.CUSTOM_MANAGERS);
   private static Class<?> x509ExtendedTrustManagerClass;
-  private static final Logger logger = Logger.getLogger(ProtocolNegotiators.class.getName());
+
   static {
     try {
       x509ExtendedTrustManagerClass = Class.forName("javax.net.ssl.X509ExtendedTrustManager");
     } catch (ClassNotFoundException e) {
-      logger.info("javax.net.ssl.X509ExtendedTrustManager is not available. Authority override via call options" +
-              "via call options will not be allowed.");
+      // Will disallow per-rpc authority override via call option.
     }
   }
 
@@ -591,17 +590,20 @@ protected void userEventTriggered0(ChannelHandlerContext ctx, Object evt) throws
 
   static final class ClientTlsProtocolNegotiator implements ProtocolNegotiator {
     private static final Logger logger = Logger.getLogger(ClientTlsProtocolNegotiator.class.getName());
-    private static Method checkServerTrustedMethod;
+    private static final Method checkServerTrustedMethod;
     static {
+      Method method = null;
       try {
         Class<?> x509ExtendedTrustManagerClass = Class.forName("javax.net.ssl.X509ExtendedTrustManager");
-        checkServerTrustedMethod = x509ExtendedTrustManagerClass.getMethod("checkServerTrusted",
+        method = x509ExtendedTrustManagerClass.getMethod("checkServerTrusted",
                 X509Certificate[].class, String.class, SSLEngine.class);
       } catch (ClassNotFoundException e) {
       } catch (NoSuchMethodException e) {
         // Should never happen.
-        logger.warning("Method checkServerTrusted not found in javax.net.ssl.X509ExtendedTrustManager");
+        logger.log(Level.WARNING, "Method checkServerTrusted not found in " +
+                "javax.net.ssl.X509ExtendedTrustManager", e);
       }
+      checkServerTrustedMethod = method;
     }
 
     @GuardedBy("this")
@@ -669,10 +671,13 @@ public synchronized Status verifyAuthority(@Nonnull String authority) {
         try {
           verifyAuthorityAllowedForPeerCert(authority);
           peerVerificationStatus = Status.OK;
-        } catch (SSLPeerUnverifiedException | CertificateException | InvocationTargetException | IllegalAccessException e) {
+        } catch (SSLPeerUnverifiedException | CertificateException | InvocationTargetException |
+                 IllegalAccessException | IllegalStateException e) {
           peerVerificationStatus = Status.UNAVAILABLE.withDescription(
                   String.format("Peer hostname verification during rpc failed for authority '%s'",
                   authority)).withCause(e);
+          logger.log(Level.WARNING, "Authority verification failed (this will be an error in the "
+                  + "future).", e);
         }
         peerVerificationResults.put(authority, peerVerificationStatus);
         return peerVerificationStatus;