Remove CallContext.getAuthenticatedPrincipal() (#1400)

Author: adutra

polaris-core/src/main/java/org/apache/polaris/core/context/CallContext.java

@@ -26,7 +26,6 @@
 import org.apache.iceberg.io.CloseableGroup;
 import org.apache.polaris.core.PolarisCallContext;
 import org.apache.polaris.core.PolarisDiagnostics;
-import org.apache.polaris.core.auth.AuthenticatedPolarisPrincipal;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -45,9 +44,6 @@ public interface CallContext extends AutoCloseable {
   // created, scoped to the current call context.
   String REQUEST_PATH_CATALOG_INSTANCE_KEY = "REQUEST_PATH_CATALOG_INSTANCE";
 
-  // Authenticator filters should populate this field alongside resolving a SecurityContext.
-  // Value type: AuthenticatedPolarisPrincipal
-  String AUTHENTICATED_PRINCIPAL = "AUTHENTICATED_PRINCIPAL";
   String CLOSEABLES = "closeables";
 
   static CallContext setCurrentContext(CallContext context) {
@@ -63,11 +59,6 @@ static PolarisDiagnostics getDiagnostics() {
     return CURRENT_CONTEXT.get().getPolarisCallContext().getDiagServices();
   }
 
-  static AuthenticatedPolarisPrincipal getAuthenticatedPrincipal() {
-    return (AuthenticatedPolarisPrincipal)
-        CallContext.getCurrentContext().contextVariables().get(CallContext.AUTHENTICATED_PRINCIPAL);
-  }
-
   static void unsetCurrentContext() {
     CURRENT_CONTEXT.remove();
   }

service/common/src/main/java/org/apache/polaris/service/auth/BasePolarisAuthenticator.java

@@ -111,8 +111,6 @@ protected Optional<AuthenticatedPolarisPrincipal> getPrincipal(DecodedToken toke
 
     AuthenticatedPolarisPrincipal authenticatedPrincipal =
         new AuthenticatedPolarisPrincipal(new PrincipalEntity(principal), activatedPrincipalRoles);
-    LOGGER.debug("Populating authenticatedPrincipal into CallContext: {}", authenticatedPrincipal);
-    callContext.contextVariables().put(CallContext.AUTHENTICATED_PRINCIPAL, authenticatedPrincipal);
     return Optional.of(authenticatedPrincipal);
   }
 }