Remove unnecessary local variable in getAuthMechanism.

rmehta19 · rmehta19 · commit 12586b11508f · 2024-09-12T09:33:04.000-07:00
diff --git a/s2a/src/main/java/io/grpc/s2a/handshaker/GetAuthenticationMechanisms.java b/s2a/src/main/java/io/grpc/s2a/handshaker/GetAuthenticationMechanisms.java
@@ -33,27 +33,23 @@ final class GetAuthenticationMechanisms {
    * @return an {@link AuthenticationMechanism} for the given local identity.
    */
   static Optional<AuthenticationMechanism> getAuthMechanism(Optional<S2AIdentity> localIdentity) {
-    Optional<AuthenticationMechanism> authMechanism = Optional.empty();
     if (!TOKEN_MANAGER.isPresent()) {
       return Optional.empty();
     }
     AccessTokenManager manager = TOKEN_MANAGER.get();
     // If no identity is provided, fetch the default access token and DO NOT attach an identity
     // to the request.
     if (!localIdentity.isPresent()) {
-      authMechanism =
-          Optional.of(
+      return Optional.of(
               AuthenticationMechanism.newBuilder().setToken(manager.getDefaultToken()).build());
     } else {
       // Fetch an access token for the provided identity.
-      authMechanism =
-          Optional.of(
+      return Optional.of(
               AuthenticationMechanism.newBuilder()
                   .setIdentity(localIdentity.get().getIdentity())
                   .setToken(manager.getToken(localIdentity.get()))
                   .build());
     }
-    return authMechanism;
   }
 
   private GetAuthenticationMechanisms() {}
