This is an automated email from the ASF dual-hosted git repository. adutra pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/polaris.git
from 5a7686bdf Rat-check: exclude venv, cleanup excludes, include .svg (#2363) add 96f1459de Refactor Authenticator and PolarisPrincipal (#2307) No new revisions were added by this update. Summary of changes: .../core/auth/AuthenticatedPolarisPrincipal.java | 63 --------------- .../polaris/core/auth/PolarisAuthorizer.java | 4 +- .../polaris/core/auth/PolarisAuthorizerImpl.java | 37 +++++---- .../apache/polaris/core/auth/PolarisPrincipal.java | 89 ++++++++++++++++++++++ .../resolver/PolarisResolutionManifest.java | 4 +- .../core/persistence/resolver/Resolver.java | 17 ++--- .../polaris/core/persistence/BaseResolverTest.java | 6 +- runtime/service/build.gradle.kts | 3 + .../polaris/service/admin/PolarisAdminService.java | 33 ++++---- .../polaris/service/admin/PolarisServiceImpl.java | 5 +- .../polaris/service/auth/ActiveRolesAugmentor.java | 7 +- .../polaris/service/auth/ActiveRolesProvider.java | 4 +- .../service/auth/AuthenticatingAugmentor.java | 23 +++--- .../apache/polaris/service/auth/Authenticator.java | 25 +++--- .../apache/polaris/service/auth/DecodedToken.java | 4 +- .../service/auth/DefaultActiveRolesProvider.java | 22 ++++-- .../polaris/service/auth/DefaultAuthenticator.java | 15 ++-- .../service/auth/PersistedPolarisPrincipal.java | 66 ++++++++++++++++ ...incipalAuthInfo.java => PolarisCredential.java} | 28 ++++--- .../service/auth/external/OidcConfiguration.java | 1 + ...or.java => OidcPolarisCredentialAugmentor.java} | 29 +++---- .../external/mapping/DefaultPrincipalMapper.java | 2 +- .../mapping/DefaultPrincipalRolesMapper.java | 2 +- .../external/tenant/DefaultOidcTenantResolver.java | 1 - .../{ => tenant}/OidcTenantConfiguration.java | 2 +- .../auth/external/tenant/OidcTenantResolver.java | 1 - .../{ => tenant}/OidcTenantResolvingAugmentor.java | 8 +- .../internal/InternalAuthenticationMechanism.java | 33 +------- .../internal/InternalAuthenticationRequest.java | 18 +++-- .../auth/internal/InternalIdentityProvider.java | 24 ++---- .../service/catalog/common/CatalogAdapter.java | 4 +- .../service/catalog/common/CatalogHandler.java | 23 +++--- .../catalog/iceberg/IcebergCatalogAdapter.java | 5 +- .../catalog/iceberg/IcebergCatalogHandler.java | 2 +- .../catalog/policy/PolicyCatalogHandler.java | 6 +- .../polaris/service/config/ServiceProducers.java | 9 +-- .../context/catalog/CallContextCatalogFactory.java | 4 +- .../catalog/PolarisCallContextCatalogFactory.java | 4 +- .../service/admin/ManagementServiceTest.java | 4 +- .../admin/PolarisAdminServiceAuthzTest.java | 8 +- .../service/admin/PolarisAdminServiceTest.java | 4 +- .../service/admin/PolarisAuthzTestBase.java | 26 ++++--- .../service/auth/ActiveRolesAugmentorTest.java | 6 +- .../service/auth/AuthenticatingAugmentorTest.java | 34 ++------- .../auth/AuthenticationConfigurationTest.java | 7 +- ...ava => OidcPolarisCredentialAugmentorTest.java} | 19 ++--- .../external/OidcTenantResolvingAugmentorTest.java | 4 +- .../mapping/DefaultPrincipalMapperTest.java | 6 +- .../mapping/DefaultPrincipalRolesMapperTest.java | 4 +- .../tenant/DefaultOidcTenantResolverTest.java | 1 - .../InternalAuthenticationMechanismTest.java | 7 +- .../internal/InternalIdentityProviderTest.java | 24 ++---- .../catalog/AbstractIcebergCatalogTest.java | 5 +- .../catalog/AbstractIcebergCatalogViewTest.java | 5 +- .../AbstractPolarisGenericTableCatalogTest.java | 6 +- .../service/catalog/AbstractPolicyCatalogTest.java | 6 +- .../catalog/IcebergCatalogHandlerAuthzTest.java | 24 +++--- ...PolarisGenericTableCatalogHandlerAuthzTest.java | 8 +- .../catalog/PolicyCatalogHandlerAuthzTest.java | 8 +- .../org/apache/polaris/service/TestServices.java | 8 +- site/content/in-dev/unreleased/external-idp.md | 6 +- site/content/in-dev/unreleased/helm.md | 5 +- 62 files changed, 435 insertions(+), 433 deletions(-) delete mode 100644 polaris-core/src/main/java/org/apache/polaris/core/auth/AuthenticatedPolarisPrincipal.java create mode 100644 polaris-core/src/main/java/org/apache/polaris/core/auth/PolarisPrincipal.java create mode 100644 runtime/service/src/main/java/org/apache/polaris/service/auth/PersistedPolarisPrincipal.java rename runtime/service/src/main/java/org/apache/polaris/service/auth/{PrincipalAuthInfo.java => PolarisCredential.java} (67%) rename runtime/service/src/main/java/org/apache/polaris/service/auth/external/{PrincipalAuthInfoAugmentor.java => OidcPolarisCredentialAugmentor.java} (79%) rename runtime/service/src/main/java/org/apache/polaris/service/auth/external/{ => tenant}/OidcTenantConfiguration.java (98%) rename runtime/service/src/main/java/org/apache/polaris/service/auth/external/{ => tenant}/OidcTenantResolvingAugmentor.java (90%) copy polaris-core/src/main/java/org/apache/polaris/core/policy/exceptions/NoSuchPolicyException.java => runtime/service/src/main/java/org/apache/polaris/service/auth/internal/InternalAuthenticationRequest.java (63%) rename runtime/service/src/test/java/org/apache/polaris/service/auth/external/{PrincipalAuthInfoAugmentorTest.java => OidcPolarisCredentialAugmentorTest.java} (86%)