[incubator-devlake] branch release-v0.12 updated: fix: search jira account by key instead of username

[zhangliang2022]

This is an automated email from the ASF dual-hosted git repository.

zhangliang2022 pushed a commit to branch release-v0.12
in repository https://gitbox.apache.org/repos/asf/incubator-devlake.git


The following commit(s) were added to refs/heads/release-v0.12 by this push:
     new 4e238ef9 fix: search jira account by key instead of username
4e238ef9 is described below

commit 4e238ef944e3a5b9c3c397fdc21a71d51afed78b
Author: zhangliang <liang.zh...@merico.dev>
AuthorDate: Tue Sep 27 18:37:49 2022 +0800

    fix: search jira account by key instead of username
---
 plugins/jira/tasks/account_collector.go | 24 +++++++-----------------
 1 file changed, 7 insertions(+), 17 deletions(-)

diff --git a/plugins/jira/tasks/account_collector.go 
b/plugins/jira/tasks/account_collector.go
index cf0036da..b6c9f39a 100644
--- a/plugins/jira/tasks/account_collector.go
+++ b/plugins/jira/tasks/account_collector.go
@@ -59,8 +59,7 @@ func CollectAccounts(taskCtx core.SubTaskContext) error {
        queryKey := "accountId"
        urlTemplate := "api/2/user"
        if data.JiraServerInfo.DeploymentType == models.DeploymentServer {
-               queryKey = "username"
-               urlTemplate = "api/2/user/search"
+               queryKey = "key"
        }
 
        collector, err := helper.NewApiCollector(helper.ApiCollectorArgs{
@@ -82,23 +81,14 @@ func CollectAccounts(taskCtx core.SubTaskContext) error {
                        return query, nil
                },
                ResponseParser: func(res *http.Response) ([]json.RawMessage, 
error) {
-                       if data.JiraServerInfo.DeploymentType == 
models.DeploymentServer {
-                               var results []json.RawMessage
-                               err := helper.UnmarshalResponse(res, &results)
-                               if err != nil {
-                                       return nil, err
-                               }
-
-                               return results, nil
-                       } else {
-                               var result json.RawMessage
-                               err := helper.UnmarshalResponse(res, &result)
-                               if err != nil {
-                                       return nil, err
-                               }
-                               return []json.RawMessage{result}, nil
+                       var result json.RawMessage
+                       err := helper.UnmarshalResponse(res, &result)
+                       if err != nil {
+                               return nil, err
                        }
+                       return []json.RawMessage{result}, nil
                },
+               AfterResponse: ignoreHTTPStatus404,
        })
        if err != nil {
                logger.Error("collect account error:", err)