Fix the bug that DSS cannot jump into the external system without login.

Author: wushengyeyouya

dss-framework/dss-framework-workspace-server/src/main/java/com/webank/wedatasphere/dss/framework/workspace/restful/WorkspaceRestfulApi.java

@@ -160,9 +160,10 @@ public Message getWorkspaceAppConns(HttpServletRequest req, @PathVariable("works
         String header = req.getHeader("Content-language").trim();
         boolean isChinese = "zh-CN".equals(header);
         String username = SecurityFilter.getLoginUsername(req);
+        Workspace workspace = SSOHelper.getWorkspace(req);
         List<WorkspaceMenuVo> appconns;
         try {
-            appconns = dssWorkspaceService.getWorkspaceAppConns(workspaceId, username, isChinese);
+            appconns = dssWorkspaceService.getWorkspaceAppConns(workspace, workspaceId, username, isChinese);
         } catch (DSSErrorException e) {
             LOGGER.warn("{} get appconns from workspace {} failed.", username, workspaceId, e);
             return Message.error(e);

dss-framework/dss-framework-workspace-server/src/main/java/com/webank/wedatasphere/dss/framework/workspace/service/DSSWorkspaceService.java

@@ -72,7 +72,8 @@ List<DSSWorkspaceUserVO> getWorkspaceUsersByRole(int workspaceId, String roleNam
 
     List<WorkspaceDepartmentVo> getWorkSpaceDepartments();
 
-    List<WorkspaceMenuVo> getWorkspaceAppConns(Long workspaceId, String username, boolean isChinese) throws DSSErrorException;
+    List<WorkspaceMenuVo> getWorkspaceAppConns(Workspace workspace, Long workspaceId,
+                                               String username, boolean isChinese) throws DSSErrorException;
 
     DSSWorkspace getWorkspacesById(Long id, String username) throws DSSErrorException;
 

dss-framework/dss-framework-workspace-server/src/main/java/com/webank/wedatasphere/dss/framework/workspace/service/impl/DSSWorkspaceServiceImpl.java

@@ -19,6 +19,7 @@
 import com.github.pagehelper.PageHelper;
 import com.github.pagehelper.PageInfo;
 import com.webank.wedatasphere.dss.appconn.core.AppConn;
+import com.webank.wedatasphere.dss.appconn.core.ext.OnlySSOAppConn;
 import com.webank.wedatasphere.dss.appconn.manager.AppConnManager;
 import com.webank.wedatasphere.dss.appconn.manager.utils.AppInstanceConstants;
 import com.webank.wedatasphere.dss.common.exception.DSSErrorException;
@@ -44,7 +45,9 @@
 import com.webank.wedatasphere.dss.framework.workspace.util.WorkspaceDBHelper;
 import com.webank.wedatasphere.dss.framework.workspace.util.WorkspaceServerConstant;
 import com.webank.wedatasphere.dss.standard.app.sso.Workspace;
+import com.webank.wedatasphere.dss.standard.app.sso.builder.SSOUrlBuilderOperation;
 import com.webank.wedatasphere.dss.standard.common.desc.AppInstance;
+import com.webank.wedatasphere.dss.standard.sso.utils.SSOHelper;
 import org.apache.commons.lang.ArrayUtils;
 import org.apache.commons.lang.StringUtils;
 import org.apache.linkis.common.exception.ErrorException;
@@ -546,7 +549,7 @@ public List<WorkspaceDepartmentVo> getWorkSpaceDepartments() {
     }
 
     private List<WorkspaceMenuVo> getMenuAppInstances(List<WorkspaceMenuVo> menuVos, List<Long> userMenuAppConnIds,
-                                                      DSSWorkspace workspace,
+                                                      DSSWorkspace dssworkspace, Workspace workspace,
                                                       boolean isChinese) {
         for (WorkspaceMenuVo menuVo : menuVos) {
             Long menuId = menuVo.getId();
@@ -556,11 +559,19 @@ private List<WorkspaceMenuVo> getMenuAppInstances(List<WorkspaceMenuVo> menuVos,
                 menuAppconn.setAccessable(userMenuAppConnIds.contains(menuAppconn.getId()));
                 AppConn appConn = AppConnManager.getAppConnManager().getAppConn(menuAppconn.getName());
                 List<DSSApplicationBean> instanceList = new ArrayList<>();
+                SSOUrlBuilderOperation operation;
+                if(appConn instanceof OnlySSOAppConn) {
+                    operation = ((OnlySSOAppConn) appConn).getOrCreateSSOStandard().getSSOBuilderService().createSSOUrlBuilderOperation();
+                    SSOHelper.setSSOUrlBuilderOperation(operation, workspace);
+                    operation.setAppName(appConn.getAppDesc().getAppName());
+                } else {
+                    operation = null;
+                }
                 appConn.getAppDesc().getAppInstances().forEach(appInstance -> {
                     String label = String.join(",", appInstance.getLabels().stream()
                             .map(l -> ((EnvDSSLabel) l).getEnv()).toArray(String[]::new));
                     String selectedName = getAppInstanceTitle(appConn, appInstance, isChinese);
-                    String homepageUri = AppInstanceConstants.getHomepageUrl(appInstance, (long) workspace.getId(), workspace.getName());
+                    String homepageUri = AppInstanceConstants.getHomepageUrl(appInstance, operation, (long) dssworkspace.getId(), dssworkspace.getName());
                     instanceList.add(new DSSApplicationBean(selectedName, appInstance.getBaseUrl(),
                             homepageUri, label));
                 });
@@ -580,12 +591,12 @@ protected String getAppInstanceTitle(AppConn appConn, AppInstance appInstance, b
     }
 
     @Override
-    public List<WorkspaceMenuVo> getWorkspaceAppConns(Long workspaceId, String username,
+    public List<WorkspaceMenuVo> getWorkspaceAppConns(Workspace workspace, Long workspaceId, String username,
                                                       boolean isChinese) throws DSSErrorException {
         DSSWorkspace dssWorkspace = getWorkspacesById(workspaceId, username);
         List<WorkspaceMenuVo> appconnMenuVos = isChinese ? workspaceMapper.getAppConnMenuCn() : workspaceMapper.getAppConnMenuEn();
         List<Long> userMenuAppConnIds = dssWorkspaceMapper.getUserMenuAppConnId(username, workspaceId);
-        return getMenuAppInstances(appconnMenuVos, userMenuAppConnIds, dssWorkspace, isChinese);
+        return getMenuAppInstances(appconnMenuVos, userMenuAppConnIds, dssWorkspace, workspace, isChinese);
     }
 
     @Override

dss-standard/dss-standard-common/src/main/java/com/webank/wedatasphere/dss/standard/common/desc/AppInstance.java

@@ -46,7 +46,11 @@ public interface AppInstance {
      * 与 dss_appinstance 数据库表的 enhance_json 字段对应，允许用户在数据库表中传入一个 json 格式的参数列表，
      * 第三方应用 AppConn 的 Operation 可以直接调用该方法获取  enhance_json 里面的参数列表。<br/>
      * 当然，您也可以通过该 AppConn 的 appconn.properties 设置一些所需的参数。appconn.properties 文件存放在
-     * ${APPCONN_HOME}/${APPCONN_NAME}/conf 目录下
+     * ${APPCONN_HOME}/${APPCONN_NAME}/conf 目录下。<br/>
+     * 目前 DSS 本身需要的参数如下：
+     *   1. reqUri：(非必须) 如果第三方应用集成了 DSS 一级规范，当在 DSS 的顶部菜单栏点击跳转到第三方应用的首页时，需要用到
+     * 该属性。 reqUri 用于指定一个 RESTFul 请求的 URI，该 URI 可访问第三方系统后台的某个 RESTFUL 接口（随意的 Restful 接口），
+     * DSS 放置在第三方系统的一级规范 Jar 包会自动拦截该请求，加上用户态后自动重定向给实际的前端首页。更多请参考 DSS 一级规范。
      * @return 第三方 AppConn 参数列表
      */
     Map<String, Object> getConfig();