René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"ZipFile.extractall fails if zipfile.Path object is created based on the ZipFile object","articleBody":"# Bug report\r\n\r\nThe following code works fine with Python 3.8 and 3.9 but starts failing with Python 3.10.\r\n\r\nIt seems that `zipfile.Path` creates an entry in the `zipfile.ZipFile` object that does not exist in the underlying file and therefore makes `zipfile.ZipFile.extractall` fail.\r\n\r\n```\r\nimport zipfile\r\nimport tempfile\r\n\r\npath = tempfile.mktemp()\r\n\r\nf = zipfile.ZipFile(path, \"w\")\r\nf.writestr(\"data/test.txt\", \"test1\")\r\nf.writestr(\"data/test2.txt\", \"test2\")\r\nf.close()\r\n\r\ndest = tempfile.mkdtemp()\r\n\r\np = zipfile.ZipFile(path)\r\nsrc_zipfile_path = zipfile.Path(p, \"data\")\r\np.extractall(dest)\r\n```\r\n\r\n# Your environment\r\n\r\nWorking version:\r\n\r\n```\r\n(python39) [moggi@mars cloudfluid]$ python --version\r\nPython 3.9.16\r\n(python39) [moggi@mars cloudfluid]$ python test.py\r\n(python39) [moggi@mars cloudfluid]$ \r\n```\r\n\r\nFailing versions:\r\n\r\n3.10.9:\r\n\r\n```\r\n(python_310_2) [moggi@mars cloudfluid]$ python --version\r\nPython 3.10.9\r\n(python_310_2) [moggi@mars cloudfluid]$ python test.py\r\nTraceback (most recent call last):\r\n  File \"/home/moggi/devel/cloudfluid/test.py\", line 15, in \u003cmodule\u003e\r\n    p.extractall(dest)\r\n  File \"/home/moggi/devel/envs/python_310_2/lib/python3.10/zipfile.py\", line 1645, in extractall\r\n    self._extract_member(zipinfo, path, pwd)\r\n  File \"/home/moggi/devel/envs/python_310_2/lib/python3.10/zipfile.py\", line 1667, in _extract_member\r\n    member = self.getinfo(member)\r\n  File \"/home/moggi/devel/envs/python_310_2/lib/python3.10/zipfile.py\", line 1441, in getinfo\r\n    raise KeyError(\r\nKeyError: \"There is no item named 'data/' in the archive\"\r\n```\r\n\r\nand 3.11.0\r\n\r\n```\r\n(python311) [moggi@mars cloudfluid]$ python --version\r\nPython 3.11.0\r\n(python311) [moggi@mars cloudfluid]$ python test.py\r\nTraceback (most recent call last):\r\n  File \"/home/moggi/devel/cloudfluid/test.py\", line 15, in \u003cmodule\u003e\r\n    p.extractall(dest)\r\n  File \"/home/moggi/devel/envs/python311/lib/python3.11/zipfile.py\", line 1677, in extractall\r\n    self._extract_member(zipinfo, path, pwd)\r\n  File \"/home/moggi/devel/envs/python311/lib/python3.11/zipfile.py\", line 1699, in _extract_member\r\n    member = self.getinfo(member)\r\n             ^^^^^^^^^^^^^^^^^^^^\r\n  File \"/home/moggi/devel/envs/python311/lib/python3.11/zipfile.py\", line 1473, in getinfo\r\n    raise KeyError(\r\nKeyError: \"There is no item named 'data/' in the archive\"\r\n\r\n```\r\n\r\nIf this is the expected new behavior I think the documentation for 3.10 should mention this breaking change and the `zipfile.Path` documentation might need an entry about this problem.\r\n\r\nThe `zipfile.Path` object is used in the original code to check if the \"data/\" directory exists in the zip file.\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-102018\n* gh-102090\n* gh-102091\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/mmohrhard","@type":"Person","name":"mmohrhard"},"datePublished":"2023-02-04T18:09:00.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":6},"url":"https://github.com/101566/cpython/issues/101566"}