Fix --version behavior to handle remote disconnect (#4269)

Author: cavcrosby

src/ansiblelint/config.py

@@ -11,6 +11,7 @@
 import warnings
 from dataclasses import dataclass, field
 from functools import lru_cache
+from http.client import HTTPException
 from importlib.metadata import PackageNotFoundError, distribution, version
 from pathlib import Path
 from typing import Any
@@ -330,7 +331,7 @@ def get_version_warning() -> str:
                 data = json.load(url)
                 with open(cache_file, "w", encoding="utf-8") as f:
                     json.dump(data, f)
-        except (URLError, HTTPError) as exc:  # pragma: no cover
+        except (URLError, HTTPError, HTTPException) as exc:  # pragma: no cover
             _logger.debug(
                 "Unable to fetch latest version from %s due to: %s",
                 release_url,

test/test_main.py

@@ -5,6 +5,7 @@
 import subprocess
 import sys
 import time
+from http.client import RemoteDisconnected
 from pathlib import Path
 
 import pytest
@@ -92,6 +93,15 @@ def test_get_version_warning_no_pip(mocker: MockerFixture) -> None:
     assert get_version_warning() == ""
 
 
+def test_get_version_warning_remote_disconnect(mocker: MockerFixture) -> None:
+    """Test that we can handle remote disconnect when fetching release url."""
+    mocker.patch("urllib.request.urlopen", side_effect=RemoteDisconnected)
+    try:
+        get_version_warning()
+    except RemoteDisconnected:
+        pytest.fail("Failed to handle a remote disconnect")
+
+
 @pytest.mark.parametrize(
     ("lintable"),
     (