Changed "sp update" work with github for future releases (#526)

* Changed "sp update" work with github for future releases

* Name changes and unified string literals

---------

Co-authored-by: Ayuto <ayuto@web.de>

Author: srpg

addons/source-python/packages/source-python/core/command/__init__.py

@@ -168,14 +168,12 @@ def update_sp(info):
     """Update Source.Python to the latest version. A restart of the server is
     required.
     """
-    core_command_logger.log_message('Please go to GitHub to check for new releases:')
-    core_command_logger.log_message('https://github.com/Source-Python-Dev-Team/Source.Python/releases')
 
-    #if not is_unversioned() and VERSION >= get_last_successful_build_number():
-    #    core_command_logger.log_message('No new version available.')
-    #    return
+    if not is_unversioned() and VERSION >= get_last_successful_build_number():
+        core_command_logger.log_message('No new version available.')
+        return
 
-    #do_full_update()
+    do_full_update()
 
 
 # =============================================================================

addons/source-python/packages/source-python/core/update.py

@@ -71,7 +71,8 @@
 DATA_URL = f'{BASE_DATA_URL}/source-python-data.zip'
 
 BASE_DOWNLOAD_URL = 'http://downloads.sourcepython.com'
-ARTIFACTS_URL = f'{BASE_DOWNLOAD_URL}/artifacts.txt'
+
+LATEST_RELEASE_URL = "https://api.github.com/repos/Source-Python-Dev-Team/Source.Python/releases/latest"
 
 DEFAULT_TIMEOUT = 3
 
@@ -144,8 +145,10 @@ def get_build_artifacts(timeout=DEFAULT_TIMEOUT):
         Number of seconds that need to pass until a timeout occurs.
     """
     update_logger.log_debug('Getting artifacts...')
-    with urlopen(ARTIFACTS_URL, timeout=timeout) as url:
-        return url.read().decode('utf-8').split('\n')
+    with urlopen(LATEST_RELEASE_URL, timeout=timeout) as url:
+        data = json.load(url)
+
+    return [asset['browser_download_url'] for asset in data.get('assets', [])]
 
 def get_download_url(game=SOURCE_ENGINE_BRANCH, timeout=DEFAULT_TIMEOUT):
     """Get the latest Source.Python download URL for a specific game.
@@ -158,9 +161,9 @@ def get_download_url(game=SOURCE_ENGINE_BRANCH, timeout=DEFAULT_TIMEOUT):
     :raise ValueError:
         Raised if the game wasn't found.
     """
-    for relative_path in get_build_artifacts(timeout):
-        if f'-{game}-' in relative_path:
-            return f'{BASE_DOWNLOAD_URL}/{relative_path}'
+    for download_url in get_build_artifacts(timeout):
+        if f'-{game}-' in download_url:
+            return download_url
 
     raise ValueError(f'Unable to find a download URL for game "{game}".')
 

addons/source-python/packages/source-python/core/version.py

@@ -6,6 +6,7 @@
 # >> IMPORTS
 # =============================================================================
 # Python Imports
+import json
 from urllib.request import urlopen
 # Source.Python Imports
 #   Cvars
@@ -32,8 +33,7 @@
 GIT_COMMIT = None
 
 LAST_SUCCESSFUL_BUILD_NUMBER_URL = (
-    'http://downloads.sourcepython.com/version.txt')
-
+    'https://api.github.com/repos/Source-Python-Dev-Team/Source.Python/releases/latest')
 
 # =============================================================================
 # >> GLOBAL VARIABLES
@@ -52,7 +52,9 @@ def get_last_successful_build_number(timeout=3):
     :rtype: int
     """
     with urlopen(LAST_SUCCESSFUL_BUILD_NUMBER_URL, timeout=timeout) as url:
-        return int(url.read().decode('utf-8'))
+        data = json.load(url)
+        tag_name = data['tag_name']
+        return int(tag_name[1:])
 
 def is_unversioned():
     """Return ``True`` if this Source.Python installation has no version.