Skip to content

Commit cfca07e

Browse files
authored
[Fix-17995][API] Non-admin users cannot delete their own access tokens (#17997)
1 parent c3d6286 commit cfca07e

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/permission/ResourcePermissionCheckServiceImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -426,7 +426,7 @@ public List<AuthorizationType> authorizationTypes() {
426426

427427
@Override
428428
public boolean permissionCheck(int userId, String url, Logger logger) {
429-
return false;
429+
return true;
430430
}
431431

432432
@Override

dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/permission/AccessTokenResourcePermissionCheckTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public class AccessTokenResourcePermissionCheckTest {
5252
@Test
5353
public void testPermissionCheck() {
5454
User user = getLoginUser();
55-
Assertions.assertFalse(accessTokenResourcePermissionCheck.permissionCheck(user.getId(), null, logger));
55+
Assertions.assertTrue(accessTokenResourcePermissionCheck.permissionCheck(user.getId(), null, logger));
5656
}
5757

5858
@Test

0 commit comments

Comments
 (0)