Skip to content

bpo-38945: UU Encoding: Removed newline characters from uu encoding methods #17418

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Dec 2, 2019
Merged

bpo-38945: UU Encoding: Removed newline characters from uu encoding methods #17418

merged 4 commits into from
Dec 2, 2019

Conversation

stealthcopter
Copy link
Contributor

@stealthcopter stealthcopter commented Nov 30, 2019 •
edited by bedevere-bot
Loading

@gvanrossum as discussed via email here is the PR for the uu encoding bug.

Sanitized filenames in UU encoding methods so that the filename cannot be overflowing in the UU encoded content area.

https://bugs.python.org/issue38945

@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@stealthcopter

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

isidentical
isidentical reviewed Nov 30, 2019
View reviewed changes
Copy link
Member

@isidentical isidentical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIK This PR needs tests and a BPO issue

@gvanrossum
Copy link
Member

gvanrossum commented Nov 30, 2019 via email •
edited
Loading

@gvanrossum
Copy link
Member

I take it back, it needs both a bpo issue and tests.

@gvanrossum
Copy link
Member

This is a security issue and needs backports to all live versions. I expect that the 2.7 backport will need some manual help, as the code is superficially different there.

@stealthcopter stealthcopter changed the title UU Encoding: Removed newline characters from uu encoding methods bpo-38945: UU Encoding: Removed newline characters from uu encoding methods Dec 1, 2019
gvanrossum
gvanrossum approved these changes Dec 1, 2019
View reviewed changes
Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Waiting for CLA and news blurb.

@stealthcopter: Can you add a news blurb per instructions from the failing test? (If you can't let me know and I can see if I can remember how to do it.)

@gvanrossum
Copy link
Member

@stealthcopter Are you sure you submitted the CLA properly?

@stealthcopter
Copy link
Contributor Author

@gvanrossum I just clicked on the 'check yourself' link and it looks like it kicked it into working :)

@gvanrossum gvanrossum merged commit a62ad47 into python:master Dec 2, 2019
@miss-islington
Copy link
Contributor

Thanks @stealthcopter for the PR, and @gvanrossum for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6, 3.7, 3.8.
🐍🍒⛏🤖

@bedevere-bot
Copy link

@gvanrossum: Please replace # with GH- in the commit message next time. Thanks!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 2, 2019
@stealthcopter @miss-islington
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
bc65cb8 
…put format (pythonGH-17418)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
@bedevere-bot
Copy link

GH-17442 is a backport of this pull request to the 3.8 branch.

@bedevere-bot
Copy link

GH-17443 is a backport of this pull request to the 3.7 branch.

@gvanrossum
Copy link
Member

Thanks! I didn't even know that that feature existed. :-) Now on to the backports.

@bedevere-bot
Copy link

GH-17444 is a backport of this pull request to the 3.6 branch.

@miss-islington
Copy link
Contributor

Sorry, @stealthcopter and @gvanrossum, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker a62ad4730c9b575f140f24074656c0257c86a09a 2.7

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 2, 2019
@stealthcopter @miss-islington
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
ffb97d1 
…put format (pythonGH-17418)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 2, 2019
@stealthcopter @miss-islington
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
fdcf8cc 
…put format (pythonGH-17418)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
@gvanrossum
Copy link
Member

@stealthcopter Can you try the 2.7 backport? I think it's simple enough, there's an unrelated change on a nearby line.

@stealthcopter
Copy link
Contributor Author

@gvanrossum yup no problem, I'll get this done tomorrow.

miss-islington added a commit that referenced this pull request Dec 2, 2019
@miss-islington @stealthcopter
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
87f2d26 
…put format (GH-17418)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Dec 2, 2019
@miss-islington @stealthcopter
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
8859fc6 
…put format (GH-17418)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
ned-deily pushed a commit that referenced this pull request Dec 2, 2019
@miss-islington @stealthcopter @ned-deily
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
30afc91 
…put format (GH-17418) (GH-17444)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
ned-deily pushed a commit to ned-deily/cpython that referenced this pull request Dec 3, 2019
@miss-islington @stealthcopter @ned-deily
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
e60c8a4 
…put format (pythonGH-17418) (pythonGH-17444)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
@sthagen sthagen mentioned this pull request Dec 3, 2019
Merged
@bedevere-bot
Copy link

GH-17452 is a backport of this pull request to the 2.7 branch.

@stealthcopter
Copy link
Contributor Author

Ok, think I've done it all now, sorry for the delay I had a bit of a battle with git.

gvanrossum pushed a commit that referenced this pull request Dec 3, 2019
@stealthcopter @gvanrossum
[2.7] bpo-38945: UU Encoding: Don't let newline in filename corrupt t…
a016d4e 
…he output format (GH-17418). (#17452)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
@stealthcopter
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
2d66da5 
…put format (python#17418)
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
@stealthcopter @shihai1991
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
db96788 
…put format (python#17418)
larryhastings pushed a commit that referenced this pull request Mar 21, 2020
@ned-deily @stealthcopter
bpo-38945: UU Encoding: Don't let newline in filename corrupt the out…
8835f46 
…put format (GH-17418) (GH-17444) (#17445)

(cherry picked from commit a62ad47)

Co-authored-by: Matthew Rollings <1211162+stealthcopter@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@isidentical isidentical isidentical left review comments

@gvanrossum gvanrossum gvanrossum approved these changes

Assignees

@gvanrossum gvanrossum

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@stealthcopter @the-knights-who-say-ni @gvanrossum @miss-islington @bedevere-bot @isidentical