Fixed OPA baseUri path

trinodb · Aug 12, 2024 · 2a28209 · 2a28209
1 parent a41fe9d
commit 2a28209
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...oxy/src/main/java/io/trino/aws/proxy/server/security/opa/OpaS3SecurityFacadeProvider.java b/...oxy/src/main/java/io/trino/aws/proxy/server/security/opa/OpaS3SecurityFacadeProvider.java
@@ -52,7 +52,7 @@ public OpaS3SecurityFacadeProvider(@ForOpa HttpClient httpClient, OpaS3SecurityM
     {
         this.httpClient = requireNonNull(httpClient, "httpClient is null");
         this.opaS3SecurityMapper = requireNonNull(opaS3SecurityMapper, "opaS3SecurityMapper is null");
-        opaServerUri = UriBuilder.fromUri(config.getOpaServerBaseUri()).path("/v1/data").build();
+        opaServerUri = UriBuilder.fromUri(config.getOpaServerBaseUri()).build();
     }
 
     @Override

diff --git a/trino-aws-proxy/src/test/java/io/trino/aws/proxy/server/TestOpaSecurity.java b/trino-aws-proxy/src/test/java/io/trino/aws/proxy/server/TestOpaSecurity.java
@@ -61,7 +61,7 @@ public static class Filter
         public TestingTrinoAwsProxyServer.Builder filter(TestingTrinoAwsProxyServer.Builder builder)
         {
             return builder.withProperty("s3-security.type", OPA_S3_SECURITY_IDENTIFIER)
-                    .withProperty("opa-s3-security.server-base-uri", "http://localhost")
+                    .withProperty("opa-s3-security.server-base-uri", "http://localhost:%s/v1/data")
                     .addModule(binder -> binder.bind(OpaS3SecurityMapper.class).to(OpaMapper.class).in(Scopes.SINGLETON))
                     .withS3Container()
                     .withOpaContainer();