libmozdata 0.2.0__tar.gz → 0.2.3__tar.gz

{libmozdata-0.2.0/libmozdata.egg-info → libmozdata-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 1.1
 Name: libmozdata
-Version: 0.2.0
+Version: 0.2.3
 Summary: Library to access and aggregate several Mozilla data sources.
 Home-page: https://github.com/mozilla/libmozdata
 Author: Mozilla Release Management

libmozdata-0.2.3/VERSION

@@ -0,0 +1 @@
+0.2.3

{libmozdata-0.2.0 → libmozdata-0.2.3}/libmozdata/bugzilla.py

@@ -7,6 +7,7 @@ import re
 
 import requests
 import six
+from requests import HTTPError
 
 import libmozdata.versions
 
@@ -1057,6 +1058,14 @@ class BugzillaShorten(Connection):
         if not self.url_handler.isactive():
             return
 
+        if not isinstance(res, dict):
+            # This is a workaround to handle cases where Bugzilla returns 200
+            # response with HTML content instead of 4xx error.
+            # See https://github.com/mozilla/libmozdata/issues/227
+            if self.RAISE_ERROR:
+                raise HTTPError("The response is not a valid JSON")
+            return
+
         self.url_handler.handle(res["url"])
 
 

{libmozdata-0.2.0 → libmozdata-0.2.3}/libmozdata/fx_trains.py

@@ -14,10 +14,36 @@ class FirefoxTrains:
     URL = "https://whattrainisitnow.com/api/"
     TIMEOUT = 30
 
+    _instance = None
+
+    def __init__(self, cache: bool = True) -> None:
+        """Constructor
+
+        Args:
+            cache: If True, the API responses will be cached.
+        """
+
+        self._cache = {} if cache else None
+
+    @classmethod
+    def get_instance(cls):
+        """Get the singleton instance of FirefoxTrains."""
+        if cls._instance is None:
+            cls._instance = cls()
+        return cls._instance
+
     def __get(self, path):
+        if self._cache is not None and path in self._cache:
+            return self._cache[path]
+
         resp = requests.get(self.URL + path, timeout=self.TIMEOUT)
         resp.raise_for_status()
-        return resp.json()
+        resp_json = resp.json()
+
+        if self._cache is not None:
+            self._cache[path] = resp_json
+
+        return resp_json
 
     def get_release_schedule(self, channel):
         """Get the release schedule for a given channel.

{libmozdata-0.2.0 → libmozdata-0.2.3}/libmozdata/lando.py

@@ -5,6 +5,8 @@
 
 import requests
 
+from . import config
+
 
 class LandoWarnings(object):
     """
@@ -14,6 +16,7 @@ class LandoWarnings(object):
     def __init__(self, api_url, api_key):
         self.api_url = f"{api_url}/diff_warnings"
         self.api_key = api_key
+        self.USER_AGENT = config.get("User-Agent", "name", "libmozdata")
 
     def del_warnings(self, warnings):
         """
@@ -24,7 +27,10 @@ class LandoWarnings(object):
 
             response = requests.delete(
                 f"{self.api_url}/{warning_id}",
-                headers={"X-Phabricator-API-Key": self.api_key},
+                headers={
+                    "X-Phabricator-API-Key": self.api_key,
+                    "User-Agent": self.USER_AGENT,
+                },
             )
 
             if response.status_code != 200:
@@ -42,7 +48,10 @@ class LandoWarnings(object):
                 "group": "LINT",
                 "data": {"message": warning},
             },
-            headers={"X-Phabricator-API-Key": self.api_key},
+            headers={
+                "X-Phabricator-API-Key": self.api_key,
+                "User-Agent": self.USER_AGENT,
+            },
         )
         if response.status_code != 201:
             raise Exception(
@@ -60,7 +69,10 @@ class LandoWarnings(object):
                 "diff_id": diff_id,
                 "group": "LINT",
             },
-            headers={"X-Phabricator-API-Key": self.api_key},
+            headers={
+                "X-Phabricator-API-Key": self.api_key,
+                "User-Agent": self.USER_AGENT,
+            },
         )
         if response.status_code != 200:
             raise Exception(

{libmozdata-0.2.0 → libmozdata-0.2.3}/libmozdata/phabricator.py

@@ -5,7 +5,6 @@
 
 import collections
 import enum
-import functools
 import json
 import logging
 from urllib.parse import urlencode, urlparse
@@ -13,6 +12,8 @@ from urllib.parse import urlencode, urlparse
 import hglib
 import requests
 
+from . import config
+
 HGMO_JSON_REV_URL_TEMPLATE = "https://hg.mozilla.org/mozilla-central/json-rev/{}"
 MOZILLA_PHABRICATOR_PROD = "https://phabricator.services.mozilla.com/api/"
 
@@ -44,14 +45,6 @@ class ArtifactType(enum.Enum):
     Uri = "uri"
 
 
-@functools.lru_cache(maxsize=2048)
-def revision_exists_on_central(revision):
-    url = HGMO_JSON_REV_URL_TEMPLATE.format(revision)
-    resp = requests.get(url)
-    resp.raise_for_status()
-    return resp.ok
-
-
 def revision_available(repo, revision):
     """
     Check if a revision is available on a Mercurial repo
@@ -215,6 +208,7 @@ class PhabricatorAPI(object):
     """
 
     def __init__(self, api_key, url=MOZILLA_PHABRICATOR_PROD):
+        self.USER_AGENT = config.get("User-Agent", "name", "libmozdata")
         self.api_key = api_key
         self.url = url
         assert self.url.endswith("/api/"), "Phabricator API must end with /api/"
@@ -228,6 +222,10 @@ class PhabricatorAPI(object):
         parts = urlparse(self.url)
         return parts.netloc
 
+    def get_header(self):
+        """Get the header to use each query"""
+        return {"User-Agent": self.USER_AGENT}
+
     def search_diffs(
         self,
         diff_phid=None,
@@ -662,6 +660,7 @@ class PhabricatorAPI(object):
         # Run POST request on api
         response = requests.post(
             self.url + path,
+            headers=self.get_header(),
             data=urlencode({"params": json.dumps(payload), "output": "json"}),
         )
         response.raise_for_status()

{libmozdata-0.2.0 → libmozdata-0.2.3}/libmozdata/release_owners.py

@@ -48,6 +48,7 @@ def get_owners():
             "Secondary",
             "Engineering REO",
             "Release Duty",
+            "QA Owner",
             "Corresponding ESR",
             "Release Date",
         ] != table[0]:
@@ -57,7 +58,7 @@ def get_owners():
         for row in table[1:]:
             try:
                 # sometimes the date is 2019-XX-XX (when the date is not known)
-                release_date = utils.get_date_ymd(row[6])
+                release_date = utils.get_date_ymd(row[7])
             except (AssertionError, ValueError):
                 continue
 
@@ -68,7 +69,8 @@ def get_owners():
                     "secondary": row[2],
                     "engineering reo": row[3],
                     "release duty": _get_list_people(row[4]),
-                    "corresponding esr": row[5],
+                    "qa owner": row[5],
+                    "corresponding esr": row[6],
                     "release date": release_date,
                 }
             )

{libmozdata-0.2.0 → libmozdata-0.2.3/libmozdata.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 1.1
 Name: libmozdata
-Version: 0.2.0
+Version: 0.2.3
 Summary: Library to access and aggregate several Mozilla data sources.
 Home-page: https://github.com/mozilla/libmozdata
 Author: Mozilla Release Management

libmozdata-0.2.0/VERSION

@@ -1 +0,0 @@
-0.2.0