Fix Jira reindex methods and implement reindex_issue (closes #1610) (#1629)

- Fix reindex(): `if not indexing_type` was always False since it
  defaults to a non-empty string, so the type param was never sent.
  Changed to `if indexing_type`.
- Fix reindex_status(): Jira returns 303 during active reindex which
  caused TooManyRedirects. Now uses allow_redirects=False.
- Add allow_redirects parameter to rest_client.request() so callers
  can opt out of redirect following when needed.
- Implement reindex_issue() which was previously a stub (just `pass`).

Author: star7js

atlassian/jira.py

@@ -4648,7 +4648,7 @@ def reindex(
             params["indexChangeHistory"] = change_history
         if not worklogs:
             params["indexWorklogs"] = worklogs
-        if not indexing_type:
+        if indexing_type:
             params["type"] = indexing_type
         url = self.resource_url("reindex")
         return self.post(url, params=params)
@@ -4676,7 +4676,8 @@ def reindex_status(self) -> T_resp_json:
         :return:
         """
         url = self.resource_url("reindex")
-        return self.get(url)
+        response = self.request("GET", path=url, allow_redirects=False)
+        return response.json()
 
     def reindex_project(self, project_key: str) -> T_resp_json:
         return self.post(
@@ -4685,8 +4686,15 @@ def reindex_project(self, project_key: str) -> T_resp_json:
             headers=self.form_token_headers,
         )
 
-    def reindex_issue(self, list_of_: list) -> None:
-        pass
+    def reindex_issue(self, issue_ids: list) -> T_resp_json:
+        """
+        Reindex specific issues by their IDs.
+        :param issue_ids: list of issue IDs (numeric) to reindex
+        :return:
+        """
+        url = self.resource_url("reindex/issue")
+        params = {"issueId": ",".join(str(i) for i in issue_ids)}
+        return self.post(url, params=params)
 
     def index_checker(self, max_results: int = 100) -> T_resp_json:
         """

atlassian/rest_client.py

@@ -440,6 +440,7 @@ def request(
         trailing: Optional[bool] = None,
         absolute: bool = False,
         advanced_mode: bool = False,
+        allow_redirects: bool = True,
     ) -> Response:
         """
 
@@ -493,6 +494,7 @@ def request(
                 files=files,
                 proxies=self.proxies,
                 cert=self.cert,
+                allow_redirects=allow_redirects,
             )
             continue_retries = retry_handler(response)
             if continue_retries: