refactor: 用户第三方平台信息添加校验功能 (#4301)

Co-authored-by: chenjianxing <jianxing.chen@fit2cloud.com>

backend/src/main/java/io/metersphere/track/controller/IssuesController.java

@@ -10,6 +10,7 @@ import io.metersphere.commons.utils.Pager;
 import io.metersphere.log.annotation.MsAuditLog;
 import io.metersphere.track.issue.domain.PlatformUser;
 import io.metersphere.track.issue.domain.zentao.ZentaoBuild;
+import io.metersphere.track.request.testcase.AuthUserIssueRequest;
 import io.metersphere.track.request.testcase.IssuesRequest;
 import io.metersphere.track.request.testcase.IssuesUpdateRequest;
 import io.metersphere.track.service.IssuesService;
@@ -59,6 +60,11 @@ public class IssuesController {
         issuesService.testAuth(orgId, platform);
     }
 
+    @PostMapping("/user/auth")
+    public void userAuth(@RequestBody AuthUserIssueRequest authUserIssueRequest) {
+        issuesService.userAuth(authUserIssueRequest);
+    }
+
     @GetMapping("/close/{id}")
     public void closeLocalIssue(@PathVariable String id) {
         issuesService.closeLocalIssue(id);

backend/src/main/java/io/metersphere/track/issue/IssuesPlatform.java

@@ -2,6 +2,7 @@ package io.metersphere.track.issue;
 
 import io.metersphere.base.domain.IssuesDao;
 import io.metersphere.base.domain.Project;
+import io.metersphere.dto.UserDTO;
 import io.metersphere.track.dto.DemandDTO;
 import io.metersphere.track.issue.domain.PlatformUser;
 import io.metersphere.track.request.testcase.IssuesRequest;
@@ -46,6 +47,11 @@ public interface IssuesPlatform {
      */
     void testAuth();
 
+    /**
+     * 用户信息测试
+     */
+    void userAuth(UserDTO.PlatformInfo userInfo);
+
     /**
      * 获取缺陷平台项目下的相关人员
      * @return platform user list

backend/src/main/java/io/metersphere/track/issue/JiraPlatform.java

@@ -268,7 +268,18 @@ public class JiraPlatform extends AbstractIssuePlatform {
     @Override
     public void testAuth() {
         setConfig();
-        jiraClientV2.getIssueCreateMetadata();
+        jiraClientV2.auth();
+    }
+
+    @Override
+    public void userAuth(UserDTO.PlatformInfo userInfo) {
+        String config = getPlatformConfig(IssuesManagePlatform.Jira.toString());
+        JiraConfig jiraConfig = JSONObject.parseObject(config, JiraConfig.class);
+        jiraConfig.setAccount(userInfo.getJiraAccount());
+        jiraConfig.setPassword(userInfo.getJiraPassword());
+        validateConfig(jiraConfig);
+        jiraClientV2.setConfig(jiraConfig);
+        jiraClientV2.auth();
     }
 
     @Override

backend/src/main/java/io/metersphere/track/issue/LocalAbstractPlatform.java

+package io.metersphere.track.issue;
+
+import io.metersphere.base.domain.IssuesDao;
+import io.metersphere.base.domain.Project;
+import io.metersphere.dto.UserDTO;
+import io.metersphere.track.issue.domain.PlatformUser;
+import io.metersphere.track.request.testcase.IssuesRequest;
+
+import java.util.List;
+
+public abstract class LocalAbstractPlatform extends AbstractIssuePlatform {
+
+    public LocalAbstractPlatform(IssuesRequest issuesRequest) { super(issuesRequest); }
+
+    @Override
+    public void testAuth() {}
+
+    @Override
+    public void userAuth(UserDTO.PlatformInfo userInfo) {}
+
+    @Override
+    public List<PlatformUser> getPlatformUser() { return null; }
+
+    @Override
+    public void syncIssues(Project project, List<IssuesDao> tapdIssues) {}
+
+    @Override
+    String getProjectId(String projectId) { return null; }
+}

backend/src/main/java/io/metersphere/track/issue/LocalPlatform.java

@@ -2,13 +2,11 @@ package io.metersphere.track.issue;
 
 import io.metersphere.base.domain.IssuesDao;
 import io.metersphere.base.domain.IssuesWithBLOBs;
-import io.metersphere.base.domain.Project;
 import io.metersphere.commons.constants.IssuesManagePlatform;
 import io.metersphere.commons.user.SessionUser;
 import io.metersphere.commons.utils.BeanUtils;
 import io.metersphere.commons.utils.SessionUtils;
 import io.metersphere.track.dto.DemandDTO;
-import io.metersphere.track.issue.domain.PlatformUser;
 import io.metersphere.track.request.testcase.IssuesRequest;
 import io.metersphere.track.request.testcase.IssuesUpdateRequest;
 import org.apache.commons.lang3.StringUtils;
@@ -16,7 +14,7 @@ import org.apache.commons.lang3.StringUtils;
 import java.util.List;
 import java.util.UUID;
 
-public class LocalPlatform extends AbstractIssuePlatform {
+public class LocalPlatform extends LocalAbstractPlatform {
 
     protected String key = IssuesManagePlatform.Local.toString();
 
@@ -67,31 +65,4 @@ public class LocalPlatform extends AbstractIssuePlatform {
     public void deleteIssue(String id) {
         issuesMapper.deleteByPrimaryKey(id);
     }
-
-    @Override
-    public void testAuth() {
-
-    }
-
-    @Override
-    public List<PlatformUser> getPlatformUser() {
-        return null;
-    }
-
-    @Override
-    public void syncIssues(Project project, List<IssuesDao> tapdIssues) {
-
-    }
-
-    @Override
-    String getProjectId(String projectId) {
-        return null;
-    }
-
-    public void closeIssue(String issueId) {
-        IssuesWithBLOBs issues = new IssuesWithBLOBs();
-        issues.setId(issueId);
-        issues.setStatus("closed");
-        issuesMapper.updateByPrimaryKeySelective(issues);
-    }
 }

backend/src/main/java/io/metersphere/track/issue/TapdPlatform.java

@@ -159,6 +159,11 @@ public class TapdPlatform extends AbstractIssuePlatform {
         }
     }
 
+    @Override
+    public void userAuth(UserDTO.PlatformInfo userInfo) {
+        testAuth();
+    }
+
     @Override
     public List<PlatformUser> getPlatformUser() {
         List<PlatformUser> users = new ArrayList<>();

backend/src/main/java/io/metersphere/track/issue/ZentaoPlatform.java

@@ -234,6 +234,16 @@ public class ZentaoPlatform extends AbstractIssuePlatform {
         zentaoClient.login();
     }
 
+    @Override
+    public void userAuth(UserDTO.PlatformInfo userInfo) {
+        String config = getPlatformConfig(IssuesManagePlatform.Zentao.toString());
+        ZentaoConfig zentaoConfig = JSONObject.parseObject(config, ZentaoConfig.class);
+        zentaoConfig.setAccount(userInfo.getZentaoUserName());
+        zentaoConfig.setPassword(userInfo.getZentaoPassword());
+        zentaoClient.setConfig(zentaoConfig);
+        zentaoClient.login();
+    }
+
     public ZentaoConfig setConfig() {
         ZentaoConfig config = getConfig();
         zentaoClient.setConfig(config);

backend/src/main/java/io/metersphere/track/issue/client/JiraAbstractClient.java

@@ -49,9 +49,13 @@ public abstract class JiraAbstractClient extends BaseClient {
         return (JiraAddIssueResponse) getResultForObject(JiraAddIssueResponse.class, response);
     }
 
-    public String getIssueCreateMetadata() {
-        ResponseEntity<String> response = restTemplate.exchange(getBaseUrl() + "/issue/createmeta", HttpMethod.GET, getAuthHttpEntity(), String.class);
-        return (String) getResultForObject(String.class, response);
+    public void auth() {
+        try {
+            restTemplate.exchange(getBaseUrl() + "/permissions", HttpMethod.GET, getAuthHttpEntity(), String.class);
+        } catch (Exception e) {
+            LogUtil.error(e.getMessage(), e);
+            MSException.throwException(e.getMessage());
+        }
     }
 
     protected HttpEntity<MultiValueMap> getAuthHttpEntity() {

backend/src/main/java/io/metersphere/track/issue/client/ZentaoClient.java

@@ -56,8 +56,14 @@ public class ZentaoClient extends BaseClient {
         String sessionId = login();
         HttpEntity<MultiValueMap<String, Object>> requestEntity = new HttpEntity<>(paramMap, new HttpHeaders());
         RestTemplate restTemplate = new RestTemplate();
-        ResponseEntity<String> response = restTemplate.exchange(getBaseUrl() + "/api-getModel-bug-create.json?zentaosid=" + sessionId,
-                HttpMethod.POST, requestEntity, String.class);
+        ResponseEntity<String> response = null;
+        try {
+           response = restTemplate.exchange(getBaseUrl() + "/api-getModel-bug-create.json?zentaosid=" + sessionId,
+                    HttpMethod.POST, requestEntity, String.class);
+        } catch (Exception e) {
+            LogUtil.error(e.getMessage(), e);
+            MSException.throwException(e.getMessage());
+        }
         AddIssueResponse addIssueResponse = (AddIssueResponse) getResultForObject(AddIssueResponse.class, response);
         return JSONObject.parseObject(addIssueResponse.getData(), AddIssueResponse.Issue.class);
     }

backend/src/main/java/io/metersphere/track/request/testcase/AuthUserIssueRequest.java

+package io.metersphere.track.request.testcase;
+
+
+import io.metersphere.dto.UserDTO;
+import lombok.Getter;
+import lombok.Setter;
+
+@Getter
+@Setter
+public class AuthUserIssueRequest extends UserDTO.PlatformInfo {
+    private String orgId;
+    private String platform;
+}

backend/src/main/java/io/metersphere/track/service/IssuesService.java

@@ -26,6 +26,7 @@ import io.metersphere.service.ProjectService;
 import io.metersphere.track.issue.*;
 import io.metersphere.track.issue.domain.PlatformUser;
 import io.metersphere.track.issue.domain.zentao.ZentaoBuild;
+import io.metersphere.track.request.testcase.AuthUserIssueRequest;
 import io.metersphere.track.request.testcase.IssuesRequest;
 import io.metersphere.track.request.testcase.IssuesUpdateRequest;
 import org.apache.commons.collections.CollectionUtils;
@@ -484,4 +485,11 @@ public class IssuesService {
     public List<IssuesDao> relateList(IssuesRequest request) {
        return extIssuesMapper.getRelateIssues(request);
     }
+
+    public void userAuth(AuthUserIssueRequest authUserIssueRequest) {
+        IssuesRequest issuesRequest = new IssuesRequest();
+        issuesRequest.setOrganizationId(authUserIssueRequest.getOrgId());
+        AbstractIssuePlatform abstractPlatform = IssueFactory.createPlatform(authUserIssueRequest.getPlatform(), issuesRequest);
+        abstractPlatform.userAuth(authUserIssueRequest);
+    }
 }

frontend/src/business/components/settings/organization/components/JiraSetting.vue

@@ -52,6 +52,12 @@
           {{ $t('organization.integration.link_the_project_now') }}
         </router-link>
       </div>
+      <div>
+        3. {{ $t('organization.integration.use_tip_three') }}
+        <router-link :to="{name: 'PersonSetting', params: { open: true }}" style="margin-left: 5px">
+          {{ $t('organization.integration.link_the_info_now') }}
+        </router-link>
+      </div>
     </div>
   </div>
 </template>

frontend/src/business/components/settings/organization/components/TapdSetting.vue

@@ -33,6 +33,12 @@
           {{ $t('organization.integration.link_the_project_now') }}
         </router-link>
       </div>
+      <div>
+        3. {{ $t('organization.integration.use_tip_three') }}
+        <router-link :to="{name: 'PersonSetting', params: { open: true }}" style="margin-left: 5px">
+          {{ $t('organization.integration.link_the_info_now') }}
+        </router-link>
+      </div>
     </div>
   </div>
 </template>

frontend/src/business/components/settings/organization/components/ZentaoSetting.vue

@@ -36,6 +36,12 @@
           {{ $t('organization.integration.link_the_project_now') }}
         </router-link>
       </div>
+      <div>
+        3. {{ $t('organization.integration.use_tip_three') }}
+        <router-link :to="{name: 'PersonSetting', params: { open: true }}" style="margin-left: 5px">
+          {{ $t('organization.integration.link_the_info_now') }}
+        </router-link>
+      </div>
     </div>
   </div>
 </template>

frontend/src/business/components/settings/personal/JiraUserInfo.vue

@@ -10,6 +10,11 @@
       <el-input v-model="data.jiraPassword" auto-complete="new-password"
                 :placeholder="$t('organization.integration.input_api_password')" show-password/>
     </el-form-item>
+    <el-form-item >
+      <el-button type="primary" style="float: right" @click="$emit('auth', 'Jira')" size="mini">
+        {{$t('commons.validate')}}
+      </el-button>
+    </el-form-item>
   </el-form>
 </template>
 

frontend/src/business/components/settings/personal/PersonSetting.vue

@@ -34,7 +34,7 @@
     </el-card>
 
     <!--Modify personal details-->
-    <el-dialog :close-on-click-modal="false" :title="$t('member.modify_personal_info')" :visible.sync="updateVisible" width="30%"
+    <el-dialog :close-on-click-modal="false" :title="$t('member.modify_personal_info')" :visible.sync="updateVisible" width="40%"
                :destroy-on-close="true" @close="handleClose">
       <el-form :model="form" label-position="right" label-width="100px" size="small" :rules="rule"
                ref="updateUserForm">
@@ -51,9 +51,9 @@
           <el-input v-model="form.phone" autocomplete="off"/>
         </el-form-item>
       </el-form>
-      <jira-user-info v-if="hasJira" :data="currentPlatformInfo"/>
-      <tapd-user-info v-if="hasTapd" :data="currentPlatformInfo"/>
-      <zentao-user-info v-if="hasZentao" :data="currentPlatformInfo"/>
+      <jira-user-info @auth="handleAuth" v-if="hasJira" :data="currentPlatformInfo"/>
+      <tapd-user-info @auth="handleAuth" v-if="hasTapd" :data="currentPlatformInfo"/>
+      <zentao-user-info @auth="handleAuth" v-if="hasZentao" :data="currentPlatformInfo"/>
       <template v-slot:footer>
         <ms-dialog-footer
           @cancel="updateVisible = false"
@@ -86,7 +86,7 @@
 </template>
 
 <script>
-import {TokenKey} from "../../../../common/js/constants";
+import {TokenKey, ZEN_TAO} from "../../../../common/js/constants";
 import MsDialogFooter from "../../common/components/MsDialogFooter";
 import {
   getCurrentOrganizationId,
@@ -276,8 +276,24 @@ export default {
         let dataList = [];
         dataList[0] = data;
         this.tableData = dataList;
+        this.handleRouteOpen();
       })
     },
+    handleRouteOpen() {
+      let params = this.$route.params;
+      if (params.open) {
+        this.edit(this.tableData[0]);
+        params.open = false;
+      }
+    },
+    handleAuth(type) {
+      let param = {...this.currentPlatformInfo};
+      param.orgId = getCurrentOrganizationId();
+      param.platform = type
+      this.$parent.result = this.$post("issues/user/auth", param, () => {
+        this.$success(this.$t('organization.integration.verified'));
+      });
+    },
     handleClose() {
       this.form = {};
       this.ruleForm = {};

frontend/src/business/components/settings/personal/TapdUserInfo.vue

@@ -10,6 +10,11 @@
     <el-form-item :label="'Tapd 昵称'" prop="account">
       <el-input v-model="data.tapdUserName" :placeholder="$t('organization.integration.input_api_account')"/>
     </el-form-item>
+    <el-form-item >
+      <el-button type="primary" style="float: right" @click="$emit('auth', 'Tapd')" size="mini">
+        {{$t('commons.validate')}}
+      </el-button>
+    </el-form-item>
   </el-form>
 </template>
 

frontend/src/business/components/settings/personal/ZentaoUserInfo.vue

@@ -10,6 +10,11 @@
       <el-input v-model="data.zentaoPassword" auto-complete="new-password"
                 :placeholder="$t('organization.integration.input_api_password')" show-password/>
     </el-form-item>
+    <el-form-item >
+      <el-button type="primary" style="float: right" @click="$emit('auth', 'Zentao')" size="mini">
+        {{$t('commons.validate')}}
+      </el-button>
+    </el-form-item>
   </el-form>
 </template>
 

frontend/src/business/components/settings/router.js

@@ -96,6 +96,7 @@ export default {
     },
     {
       path: 'personsetting',
+      name: 'PersonSetting',
       component: () => import('@/business/components/settings/personal/PersonSetting'),
       meta: {person: true, title: 'commons.personal_setting'}
     },

frontend/src/i18n/en-US.js

@@ -399,7 +399,9 @@ export default {
       use_tip_jira: 'Jira software server authentication information is account password, Jira software cloud authentication information is account + token (account settings-security-create API token)',
       use_tip_zentao: 'The account password is a Zentao account with corresponding permissions, and the account needs to have super model calling interface permissions',
       use_tip_two: 'After saving the Basic Auth account information, you need to manually associate the ID/key and issue template in the MeterSphere project',
+      use_tip_three: 'Add your personal platform account',
       link_the_project_now: 'Link the project now',
+      link_the_info_now: 'Add now',
       cancel_edit: 'Cancel edit',
       cancel_integration: 'Cancel integration',
       cancel_confirm: 'Confirm cancellation of integration ',