Skip to content

Commit e33c220

Browse files
committed
fix ci
1 parent 168a72f commit e33c220

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

server-common/src/main/java/org/apache/gravitino/server/authorization/expression/AuthorizationExpressionConverter.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,9 @@ public static String convertToOgnlExpression(String authorizationExpression) {
9191
String privilegeOrExpression = matcher.group(2);
9292
String replacement;
9393
if (AuthConstants.OWNER.equals(privilegeOrExpression)) {
94-
replacement = String.format("authorizer.isOwner(principal,METALAKE_NAME,%s)", type);
94+
replacement =
95+
String.format(
96+
"authorizer.isOwner(principal,METALAKE_NAME,%s,authorizationContext)", type);
9597
} else if (privilegeOrExpression.startsWith(DENY_PREFIX)) {
9698
String privilege = privilegeOrExpression.substring(5);
9799
replacement =

server-common/src/test/java/org/apache/gravitino/server/authorization/expression/TestAuthorizationExpressionConverter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public void testConvertToOgnlWithOwnerExpression() {
9191
String useCatalogOgnExpression =
9292
AuthorizationExpressionConverter.convertToOgnlExpression(expressionWithOwner2);
9393
Assertions.assertEquals(
94-
"(authorizer.isOwner(principal,METALAKE_NAME,METALAKE) "
94+
"(authorizer.isOwner(principal,METALAKE_NAME,METALAKE,authorizationContext) "
9595
+ "|| authorizer.isOwner(principal,METALAKE_NAME,CATALOG,authorizationContext))"
9696
+ " && authorizer.authorize(principal,METALAKE_NAME,CATALOG"
9797
+ ",@org.apache.gravitino.authorization.Privilege$Name@USE_CATALOG,authorizationContext))",

0 commit comments

Comments
 (0)