XENG-9362 Support github api without version

Author: Jacob Truman

buildrunner/config/fetch/github.py

@@ -15,8 +15,11 @@
     BuildRunnerProtocolError,
 )
 
+def _clean_nones(items):
+    return list(filter(None, items))
 
-def v3_fetch_file(parsed_url, gh_config: GithubModel):
+
+def _fetch_file(parsed_url, gh_config: GithubModel):
     """
     Fetch files using Github v3 protocol.
     """
@@ -30,12 +33,12 @@ def v3_fetch_file(parsed_url, gh_config: GithubModel):
 
     auth = (username, gh_config.app_token)
     url = "/".join(
-        (
+        _clean_nones([
             endpoint,
             version,
             "users",
             username,
-        )
+        ])
     )
     resp = requests.get(url, auth=auth, timeout=180)
     if resp.status_code != 200:
@@ -47,7 +50,7 @@ def v3_fetch_file(parsed_url, gh_config: GithubModel):
         raise ValueError('URL must begin with "/"')
 
     fpath = parsed_url.path.split("/")
-    ubuild = [endpoint, version, "repos", fpath[1], fpath[2], "contents"]
+    ubuild = _clean_nones([endpoint, version, "repos", fpath[1], fpath[2], "contents"])
     ubuild.extend(fpath[3:])
     url = "/".join(ubuild)
     resp = requests.get(url, auth=auth, timeout=180)
@@ -97,9 +100,9 @@ def fetch_file(parsed_url, config: GlobalConfig):
         )
 
     ver = nlcfg.version
-    # NOTE: potentially the v3_fetch_file() works for other github API versions.
-    if ver == "v3":
-        contents = v3_fetch_file(parsed_url, nlcfg)
+    # NOTE: potentially the _fetch_file() works for other github API versions.
+    if ver == "v3" or ver is None:
+        contents = _fetch_file(parsed_url, nlcfg)
     else:
         raise NotImplementedError(f"No version support for github API version {ver}")
 

buildrunner/config/models.py

@@ -30,7 +30,7 @@
 
 class GithubModel(BaseModel, extra="forbid"):
     endpoint: str
-    version: str
+    version: str = None
     username: str = os.getenv("USER", os.getenv("LOGNAME"))
     app_token: str = ""
 

docs/fetching-files.rst

@@ -57,7 +57,7 @@ The ``github://`` facility requires the following configuration entries:
   github:
     LABEL:
       endpoint: 'https://HOSTNAME/API_PATH'
-      version: 'VERSION'
+      version: 'VERSION' // optional
       username: 'USERNAME'
       app_token: 'APP_TOKEN'
 
@@ -72,6 +72,16 @@ The following is suggested for an entry to reference files for GitHub Enterprise
       username: 'USERNAME'
       app_token: 'APP_TOKEN'
 
+The following is suggested for an entry to reference files for GitHub.com:
+
+.. code:: yaml
+
+  github:
+    github:
+      endpoint: 'https://api.github.com'
+      username: 'USERNAME'
+      app_token: 'APP_TOKEN'
+
 :username: The individual username used to access the Github Enterprise instance.
 :app_token: The user-specific application token generated by the user on Github for Buildrunner
             access.  It is a 40 hex digit token.