Refactor: Removed an unnecessary comment and renamed PrestoAuthentica…

…tor.java to CustomPrestoAuthenticator.java
prestodb · Dec 13, 2024 · c175051 · c175051
1 parent 245eae2
commit c175051
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.../server/security/PrestoAuthenticator.java → ...r/security/CustomPrestoAuthenticator.java b/.../server/security/PrestoAuthenticator.java → ...r/security/CustomPrestoAuthenticator.java
@@ -24,13 +24,13 @@
 
 import static java.util.Objects.requireNonNull;
 
-public class PrestoAuthenticator
+public class CustomPrestoAuthenticator
         implements Authenticator
 {
     private PrestoAuthenticatorManager authenticatorManager;
 
     @Inject
-    public PrestoAuthenticator(PrestoAuthenticatorManager authenticatorManager)
+    public CustomPrestoAuthenticator(PrestoAuthenticatorManager authenticatorManager)
     {
         this.authenticatorManager = requireNonNull(authenticatorManager, "authenticatorManager is null");
         authenticatorManager.setRequired();
@@ -41,7 +41,6 @@ public Principal authenticate(HttpServletRequest request)
             throws AuthenticationException
     {
         try {
-            //wrapped the original HttpServletRequest in a RestrictedHttpServletRequest
             ReadOnlyHttpServletRequest restrictedRequest = new ReadOnlyHttpServletRequest(request);
             return authenticatorManager.getAuthenticator().createAuthenticatedPrincipal(restrictedRequest);
         }

diff --git a/presto-main/src/main/java/com/facebook/presto/server/security/ServerSecurityModule.java b/presto-main/src/main/java/com/facebook/presto/server/security/ServerSecurityModule.java
@@ -61,7 +61,7 @@ else if (authType == JWT) {
                 authBinder.addBinding().to(JsonWebTokenAuthenticator.class).in(Scopes.SINGLETON);
             }
             else if (authType == CUSTOM) {
-                authBinder.addBinding().to(PrestoAuthenticator.class).in(Scopes.SINGLETON);
+                authBinder.addBinding().to(CustomPrestoAuthenticator.class).in(Scopes.SINGLETON);
             }
             else {
                 throw new AssertionError("Unhandled auth type: " + authType);