-
Notifications
You must be signed in to change notification settings - Fork 136
Fix leaking file descriptors in streamed download #7158
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.
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
Conversation
cfaf880 to
d281877
Compare
d281877 to
9748209
Compare
e4cd8ea to
0cd4378
Compare
ggainey
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we get a CHANGES/7157.bugfix please? Thanks!
Fixes pulp#7157 Co-authored-by: Gerrod <gerrodubben@gmail.com>
0cd4378 to
9bc2b96
Compare
|
I added a changelog and also "subscribed" to this PR, since @m-bucher is on PTO at the moment. Feel free to ping me if anything more is needed. |
|
Do we need some backports? |
Backport to 3.85: 💚 backport PR created✅ Backport PR branch: Backported as #7167 🤖 @patchback |
Backport to 3.73: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 4307aaa on top of patchback/backports/3.73/4307aaacd4169a78fde43e2a5cbf664e50c9ef2e/pr-7158 Backporting merged PR #7158 into main
🤖 @patchback |
Backport to 3.95: 💚 backport PR created✅ Backport PR branch: Backported as #7166 🤖 @patchback |
Fixes #7157
RfC ;-)