(improvement)(Headless) Fix model list auth checking (#883)

* (improvement)(Headless) Fix model list auth check

* (improvement)(Headless) Fix row permission

---------

Co-authored-by: jolunoluo

Author: lxwcodemonkey

headless/server/src/main/java/com/tencent/supersonic/headless/server/service/impl/ModelServiceImpl.java

@@ -306,11 +306,16 @@ public List<ModelResp> getModelRespAuthInheritDomain(User user, Long domainId, A
         List<Long> domainIds = domainService.getDomainAuthSet(user, authType)
                 .stream().map(DomainResp::getId)
                 .collect(Collectors.toList());
-        if (domainIds.contains(domainId)) {
-            domainIds = Lists.newArrayList(domainId);
-        } else {
+        if (CollectionUtils.isEmpty(domainIds)) {
             return Lists.newArrayList();
         }
+        if (domainId != null) {
+            if (domainIds.contains(domainId)) {
+                domainIds = Lists.newArrayList(domainId);
+            } else {
+                return Lists.newArrayList();
+            }
+        }
         ModelFilter modelFilter = new ModelFilter();
         modelFilter.setIncludesDetail(false);
         modelFilter.setDomainIds(domainIds);

launchers/standalone/src/test/java/com/tencent/supersonic/headless/SchemaAuthTest.java

@@ -39,7 +39,7 @@ public void test_getDomainList_alice() {
     @Test
     public void test_getModelList_alice() {
         User user = DataUtils.getUserAlice();
-        List<ModelResp> modelResps = modelService.getModelListWithAuth(user, 0L, AuthType.ADMIN);
+        List<ModelResp> modelResps = modelService.getModelListWithAuth(user, null, AuthType.ADMIN);
         List<Long> expectedModelIds = Lists.newArrayList(1L, 4L);
         Assertions.assertEquals(expectedModelIds,
                 modelResps.stream().map(ModelResp::getId).collect(Collectors.toList()));
@@ -48,7 +48,7 @@ public void test_getModelList_alice() {
     @Test
     public void test_getVisibleModelList_alice() {
         User user = DataUtils.getUserAlice();
-        List<ModelResp> modelResps = modelService.getModelListWithAuth(user, 0L, AuthType.VISIBLE);
+        List<ModelResp> modelResps = modelService.getModelListWithAuth(user, null, AuthType.VISIBLE);
         List<Long> expectedModelIds = Lists.newArrayList(1L, 4L);
         Assertions.assertEquals(expectedModelIds,
                 modelResps.stream().map(ModelResp::getId).collect(Collectors.toList()));
@@ -75,7 +75,7 @@ public void test_getDomainList_jack() {
     @Test
     public void test_getModelList_jack() {
         User user = DataUtils.getUserJack();
-        List<ModelResp> modelResps = modelService.getModelListWithAuth(user, 0L, AuthType.ADMIN);
+        List<ModelResp> modelResps = modelService.getModelListWithAuth(user, null, AuthType.ADMIN);
         List<Long> expectedModelIds = Lists.newArrayList(1L, 2L, 3L);
         Assertions.assertEquals(expectedModelIds,
                 modelResps.stream().map(ModelResp::getId).collect(Collectors.toList()));

launchers/standalone/src/test/resources/application-local.yaml

@@ -77,4 +77,4 @@ logging:
 
 inMemoryEmbeddingStore:
   persistent:
-    path: d://
+    path: /tmp