Merge pull request #106 from mdevolde/master

Minor bug in the detection of non-existent versions

Author: jxmorris12

README.md

@@ -270,7 +270,7 @@ into where the ``language_tool_python`` package resides.
 
 ### LanguageTool Version
 
-As of April 2020, `language_tool_python` was forked from `language-check` and no longer supports LanguageTool versions lower than 4.0.
+LanguageTool versions under 6.0 are no longer downloadable from the LanguageTool website. If you need to use an older version, you can download it from the [LanguageTool GitHub tags page](https://github.com/languagetool-org/languagetool/tags) and build it yourself.
 
 ### Acknowledgements
 

language_tool_python/download_lt.py

@@ -13,6 +13,7 @@
 from .utils import (
     find_existing_language_tool_downloads,
     get_language_tool_download_path,
+    PathError,
     LTP_JAR_DIR_PATH_ENV_VAR
 )
 
@@ -125,13 +126,13 @@ def http_get(url: str, out_file: IO[bytes], proxies: Optional[Dict[str, str]] =
     :type out_file: IO[bytes]
     :param proxies: Optional dictionary of proxies to use for the request.
     :type proxies: Optional[Dict[str, str]]
-    :raises Exception: If the file could not be found at the given URL (HTTP 403).
+    :raises PathError: If the file could not be found at the given URL (HTTP 404).
     """
     req = requests.get(url, stream=True, proxies=proxies)
     content_length = req.headers.get('Content-Length')
     total = int(content_length) if content_length is not None else None
-    if req.status_code == 403:  # Not found on AWS
-        raise Exception(f'Could not find at URL {url}.')
+    if req.status_code == 404:
+        raise PathError(f'Could not find at URL {url}. The given version may not exist or is no longer available.')
     version = re.search(r'(\d+\.\d+)', url).group(1)
     progress = tqdm.tqdm(unit="B", unit_scale=True, total=total,
                          desc=f'Downloading LanguageTool {version}')

language_tool_python/utils.py

@@ -65,7 +65,8 @@ class PathError(LanguageToolError):
     """
     Exception raised for errors in the file path used in LanguageTool.
     This error is raised when there is an issue with the file path provided
-    to LanguageTool, such as the LanguageTool JAR file not being found.
+    to LanguageTool, such as the LanguageTool JAR file not being found,
+    or a download path not being a valid available file path.
     """
     pass
 

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "language_tool_python"
-version = "2.9.0"
+version = "2.9.1"
 requires-python = ">=3.9"
 description = "Checks grammar using LanguageTool."
 readme = { file = "README.md", content-type = "text/markdown" }