dimas-b commented on code in PR #2680: URL: https://github.com/apache/polaris/pull/2680#discussion_r2379791838
########## runtime/defaults/src/main/resources/application.properties: ########## @@ -193,6 +193,15 @@ polaris.oidc.principal-roles-mapper.type=default # polaris.storage.gcp.token=token # polaris.storage.gcp.lifespan=PT1H +# Polaris authorization implementation settings +# Which authorizer to use: "default" (PolarisAuthorizerImpl) or "opa" (OpaPolarisAuthorizer) +polaris.authorization.implementation=default + +# OPA Authorizer Configuration: effective only if polaris.authorization.implementation=opa +# polaris.authorization.opa.url=http://localhost:8181 Review Comment: Does it requite authentication? ########## polaris-core/build.gradle.kts: ########## @@ -24,6 +24,7 @@ plugins { dependencies { implementation(project(":polaris-api-management-model")) + implementation("com.squareup.okhttp3:okhttp:4.12.0") Review Comment: Polaris Servers already have the Apache Http client. Would it be good enough for this authorizer? Side note: please use `gradle/libs.versions.toml` for version management. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
