info: require pkginfo >= 1.12 for METADATA 2.4 support and loosen check for unknown METADATA versions (#9888)

(cherry picked from commit a3cae0c)

Author: radoering

poetry.lock

@@ -45,7 +45,7 @@ keyring = "^24.0.0"
 # packaging uses calver, so version is unclamped
 packaging = ">=23.1"
 pexpect = "^4.7.0"
-pkginfo = "^1.10"
+pkginfo = "^1.12"
 platformdirs = ">=3.0.0,<5"
 pyproject-hooks = "^1.0.0"
 requests = "^2.26"

pyproject.toml

@@ -245,9 +245,13 @@ def _from_distribution(
 
         :param dist: The distribution instance to parse information from.
         """
-        if dist.metadata_version not in pkginfo.distribution.HEADER_ATTRS:
-            # This check can be replaced once upstream implements strict parsing
-            # https://bugs.launchpad.net/pkginfo/+bug/2058697
+        # If the METADATA version is greater than the highest supported version,
+        # pkginfo prints a warning and tries to parse the fields from the highest
+        # known version. Assuming that METADATA versions adhere to semver,
+        # this should be safe for minor updates.
+        if not dist.metadata_version or dist.metadata_version.split(".")[0] not in {
+            v.split(".")[0] for v in pkginfo.distribution.HEADER_ATTRS
+        }:
             raise ValueError(f"Unknown metadata version: {dist.metadata_version}")
 
         requirements = None

src/poetry/inspection/info.py

@@ -179,6 +179,20 @@ def test_info_from_wheel(demo_wheel: Path) -> None:
     assert info._source_url == demo_wheel.resolve().as_posix()
 
 
+@pytest.mark.parametrize("version", ["24", "299"])
+def test_info_from_wheel_metadata_versions(
+    version: str, fixture_dir: FixtureDirGetter
+) -> None:
+    path = (
+        fixture_dir("distributions")
+        / f"demo_metadata_version_{version}-0.1.0-py2.py3-none-any.whl"
+    )
+    info = PackageInfo.from_wheel(path)
+    demo_check_info(info)
+    assert info._source_type == "file"
+    assert info._source_url == path.resolve().as_posix()
+
+
 def test_info_from_wheel_metadata_version_unknown(
     fixture_dir: FixtureDirGetter,
 ) -> None: