Mailing List Archive

https://github.com/python/cpython/commit/aa26dc3c4aa148d28eb7ea910642a6d012cd8921
commit: aa26dc3c4aa148d28eb7ea910642a6d012cd8921
branch: 3.12
author: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
committer: encukou <encukou@gmail.com>
date: 2024-04-16T11:54:01Z
summary:

[3.12] gh-117691: Add an appropriate stacklevel for PEP-706 tarfile deprecation warnings (GH-117872) (GH-117930)

gh-117691: Add an appropriate stacklevel for PEP-706 tarfile deprecation warnings (GH-117872)
(cherry picked from commit cff0a2db00b6379f60fe273a9782f71773d0a4cb)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>

files:
A Misc/NEWS.d/next/Library/2024-04-14-15-59-28.gh-issue-117691.1mtREE.rst
M Lib/tarfile.py
M Lib/test/test_tarfile.py

diff --git a/Lib/tarfile.py b/Lib/tarfile.py
index 3bbbcaa6211191..e1487e3864d44b 100755
--- a/Lib/tarfile.py
+++ b/Lib/tarfile.py
@@ -2222,7 +2222,7 @@ def _get_filter_function(self, filter):
'Python 3.14 will, by default, filter extracted tar '
+ 'archives and reject files or modify their metadata. '
+ 'Use the filter argument to control this behavior.',
- DeprecationWarning)
+ DeprecationWarning, stacklevel=3)
return fully_trusted_filter
if isinstance(filter, str):
raise TypeError(
diff --git a/Lib/test/test_tarfile.py b/Lib/test/test_tarfile.py
index 9aa17267490384..414843ea4a2844 100644
--- a/Lib/test/test_tarfile.py
+++ b/Lib/test/test_tarfile.py
@@ -718,6 +718,31 @@ def test_extract_directory(self):
finally:
os_helper.rmtree(DIR)

+ def test_deprecation_if_no_filter_passed_to_extractall(self):
+ DIR = pathlib.Path(TEMPDIR) / "extractall"
+ with (
+ os_helper.temp_dir(DIR),
+ tarfile.open(tarname, encoding="iso8859-1") as tar
+ ):
+ directories = [t for t in tar if t.isdir()]
+ with self.assertWarnsRegex(DeprecationWarning, "Use the filter argument") as cm:
+ tar.extractall(DIR, directories)
+ # check that the stacklevel of the deprecation warning is correct:
+ self.assertEqual(cm.filename, __file__)
+
+ def test_deprecation_if_no_filter_passed_to_extract(self):
+ dirtype = "ustar/dirtype"
+ DIR = pathlib.Path(TEMPDIR) / "extractall"
+ with (
+ os_helper.temp_dir(DIR),
+ tarfile.open(tarname, encoding="iso8859-1") as tar
+ ):
+ tarinfo = tar.getmember(dirtype)
+ with self.assertWarnsRegex(DeprecationWarning, "Use the filter argument") as cm:
+ tar.extract(tarinfo, path=DIR)
+ # check that the stacklevel of the deprecation warning is correct:
+ self.assertEqual(cm.filename, __file__)
+
def test_extractall_pathlike_name(self):
DIR = pathlib.Path(TEMPDIR) / "extractall"
with os_helper.temp_dir(DIR), \
diff --git a/Misc/NEWS.d/next/Library/2024-04-14-15-59-28.gh-issue-117691.1mtREE.rst b/Misc/NEWS.d/next/Library/2024-04-14-15-59-28.gh-issue-117691.1mtREE.rst
new file mode 100644
index 00000000000000..d90817a9ebde2f
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2024-04-14-15-59-28.gh-issue-117691.1mtREE.rst
@@ -0,0 +1,5 @@
+Improve the error messages emitted by :mod:`tarfile` deprecation warnings
+relating to PEP 706. If a ``filter`` argument is not provided to
+``extract()`` or ``extractall``, the deprecation warning now points to the
+line in the user's code where the relevant function was called. Patch by
+Alex Waygood.

_______________________________________________
Python-checkins mailing list -- python-checkins@python.org
To unsubscribe send an email to python-checkins-leave@python.org
https://mail.python.org/mailman3/lists/python-checkins.python.org/
Member address: list-python-checkins@lists.gossamer-threads.com