[issue34165] uu.decode() raises binascii.Error instead of uu.Error on invalid data

[Irit Katriel]

Irit Katriel  added the comment:

uu is deprecated as per PEP 594, so there won't be further enhancements to it.

--
nosy: +iritkatriel
resolution:  -> wont fix
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34165] uu.decode() raises binascii.Error instead of uu.Error on invalid data

[Pulak Malhotra]

Change by Pulak Malhotra :


--
keywords: +patch
nosy: +PulakIIIT
nosy_count: 1.0 -> 2.0
pull_requests: +28850
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/30648

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34165] uu.decode() raises binascii.Error instead of uu.Error on invalid data

[Irit Katriel]

Change by Irit Katriel :


--
keywords: +easy
type:  -> behavior
versions: +Python 3.10, Python 3.11, Python 3.9 -Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34165] uu.decode() raises binascii.Error instead of uu.Error on invalid data

[Jussi Judin]

New submission from Jussi Judin :

uu.decode() function can leak the internal binascii.Error exception from 
binascii.a2b_uu() function call instead of the documented uu.Error exception.

Following code demonstrates the issue:

>>> import uu
>>> with open("in.uu", "wb") as fp:
... fp.write(b'begin 0 \n0\xe8')
>>> uu.decode("in.uu", "out.uu")
Traceback (most recent call last):
  File "/tmp/python-3.7-bin/lib/python3.7/uu.py", line 148, in decode
data = binascii.a2b_uu(s)
binascii.Error: Illegal char

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "", line 1, in 
  File "/tmp/python-3.7-bin/lib/python3.7/uu.py", line 152, in decode
data = binascii.a2b_uu(s[:nbytes])
binascii.Error: Illegal char

It looks like the the workaround for broken encoders that catches the first 
binascii.Error exception just lets the second one to propagate if the recovery 
fails. I would except uu.Error to be raised instead, as that is mentioned in 
the documentation.

--
components: Library (Lib)
messages: 321994
nosy: Barro
priority: normal
severity: normal
status: open
title: uu.decode() raises binascii.Error instead of uu.Error on invalid data
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com