chore: remove broad Exception catching from config.py

JohnVillalovos · JohnVillalovos · commit 1471c82f39ac · 2022-08-01T23:09:45.000-07:00
Change "except Exception:" catching to more granular exceptions.

A step in enabling the "broad-except" check in pylint.
diff --git a/gitlab/config.py b/gitlab/config.py
@@ -165,108 +165,102 @@ def _parse_config(self) -> None:
             # Value Error means the option exists but isn't a boolean.
             # Get as a string instead as it should then be a local path to a
             # CA bundle.
-            try:
-                self.ssl_verify = _config.get("global", "ssl_verify")
-            except Exception:  # pragma: no cover
-                pass
-        except Exception:
+            self.ssl_verify = _config.get("global", "ssl_verify")
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
         try:
             self.ssl_verify = _config.getboolean(self.gitlab_id, "ssl_verify")
         except ValueError:
             # Value Error means the option exists but isn't a boolean.
             # Get as a string instead as it should then be a local path to a
             # CA bundle.
-            try:
-                self.ssl_verify = _config.get(self.gitlab_id, "ssl_verify")
-            except Exception:  # pragma: no cover
-                pass
-        except Exception:
+            self.ssl_verify = _config.get(self.gitlab_id, "ssl_verify")
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.timeout = _config.getint("global", "timeout")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
         try:
             self.timeout = _config.getint(self.gitlab_id, "timeout")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.private_token = _config.get(self.gitlab_id, "private_token")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.oauth_token = _config.get(self.gitlab_id, "oauth_token")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.job_token = _config.get(self.gitlab_id, "job_token")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.http_username = _config.get(self.gitlab_id, "http_username")
             self.http_password = _config.get(
                 self.gitlab_id, "http_password"
             )  # pragma: no cover
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         self._get_values_from_helper()
 
         try:
             self.api_version = _config.get("global", "api_version")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
         try:
             self.api_version = _config.get(self.gitlab_id, "api_version")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
         if self.api_version not in ("4",):
             raise GitlabDataError(f"Unsupported API version: {self.api_version}")
 
         for section in ["global", self.gitlab_id]:
             try:
                 self.per_page = _config.getint(section, "per_page")
-            except Exception:
+            except (configparser.NoOptionError, configparser.NoSectionError):
                 pass
         if self.per_page is not None and not 0 <= self.per_page <= 100:
             raise GitlabDataError(f"Unsupported per_page number: {self.per_page}")
 
         try:
             self.pagination = _config.get(self.gitlab_id, "pagination")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.order_by = _config.get(self.gitlab_id, "order_by")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.user_agent = _config.get("global", "user_agent")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
         try:
             self.user_agent = _config.get(self.gitlab_id, "user_agent")
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
         try:
             self.retry_transient_errors = _config.getboolean(
                 "global", "retry_transient_errors"
             )
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
         try:
             self.retry_transient_errors = _config.getboolean(
                 self.gitlab_id, "retry_transient_errors"
             )
-        except Exception:
+        except (configparser.NoOptionError, configparser.NoSectionError):
             pass
 
     def _get_values_from_helper(self) -> None:
