diff --git a/openmetadata-mcp/src/main/java/org/openmetadata/mcp/auth/OAuthAuthorizationServerProvider.java b/openmetadata-mcp/src/main/java/org/openmetadata/mcp/auth/OAuthAuthorizationServerProvider.java index 0673b3625356..7d488307b756 100644 --- a/openmetadata-mcp/src/main/java/org/openmetadata/mcp/auth/OAuthAuthorizationServerProvider.java +++ b/openmetadata-mcp/src/main/java/org/openmetadata/mcp/auth/OAuthAuthorizationServerProvider.java @@ -75,16 +75,6 @@ CompletableFuture loadAuthorizationCode( CompletableFuture exchangeAuthorizationCode( OAuthClientInformation client, AuthorizationCode authorizationCode) throws TokenException; - /** - * Loads a RefreshToken by its token string. - * @param client The client that is requesting to load the refresh token. - * @param refreshToken The refresh token string to load. - * @return A CompletableFuture that resolves to the RefreshToken object if found, or - * null if not found. - */ - CompletableFuture loadRefreshToken( - OAuthClientInformation client, String refreshToken); - /** * Exchanges a refresh token for an access token and refresh token. * @param client The client exchanging the refresh token. @@ -98,14 +88,6 @@ CompletableFuture exchangeRefreshToken( OAuthClientInformation client, RefreshToken refreshToken, List scopes) throws TokenException; - /** - * Loads an access token by its token. - * @param token The access token to verify. - * @return A CompletableFuture that resolves to the AccessToken, or null if the token - * is invalid. - */ - CompletableFuture loadAccessToken(String token); - /** * Revokes an access or refresh token. * @param token The token to revoke. diff --git a/openmetadata-mcp/src/main/java/org/openmetadata/mcp/server/auth/provider/UserSSOOAuthProvider.java b/openmetadata-mcp/src/main/java/org/openmetadata/mcp/server/auth/provider/UserSSOOAuthProvider.java index 183e445dec16..85170cdc5f10 100644 --- a/openmetadata-mcp/src/main/java/org/openmetadata/mcp/server/auth/provider/UserSSOOAuthProvider.java +++ b/openmetadata-mcp/src/main/java/org/openmetadata/mcp/server/auth/provider/UserSSOOAuthProvider.java @@ -20,7 +20,6 @@ import java.util.Map; import java.util.concurrent.CompletableFuture; import lombok.extern.slf4j.Slf4j; -import org.openmetadata.mcp.auth.AccessToken; import org.openmetadata.mcp.auth.AuthorizationCode; import org.openmetadata.mcp.auth.AuthorizationParams; import org.openmetadata.mcp.auth.OAuthAuthorizationServerProvider; @@ -807,16 +806,6 @@ public CompletableFuture loadAuthorizationCode( } } - @Override - public CompletableFuture loadRefreshToken( - OAuthClientInformation client, String refreshToken) { - // Refresh token validation happens in exchangeRefreshToken which verifies JWT directly - // This method is not used in the current implementation - return CompletableFuture.failedFuture( - new UnsupportedOperationException( - "loadRefreshToken not implemented - use exchangeRefreshToken instead")); - } - @Override public CompletableFuture exchangeRefreshToken( OAuthClientInformation client, RefreshToken refreshToken, List scopes) @@ -940,15 +929,6 @@ public CompletableFuture exchangeRefreshToken( } } - @Override - public CompletableFuture loadAccessToken(String token) { - // Access token validation happens through JwtFilter which verifies JWT directly - // This method is not used in the current implementation - return CompletableFuture.failedFuture( - new UnsupportedOperationException( - "loadAccessToken not implemented - tokens are validated via JwtFilter")); - } - @Override public CompletableFuture revokeToken(Object token) { LOG.info("Token revocation requested");