This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author lukasz.langa
Recipients amacd31, bmerry, christian.heimes, jakirkham, jan-xyz, lukasz.langa, matan1008, methane, ronaldoussoren
Date 2021-07-29.16:08:26
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1627574906.16.0.720768349468.issue42853@roundup.psfhosted.org>
In-reply-to
Content
> This fix is going to cause a regression of bpo-36050. Would it not be possible to fix this in _ssl.c (by breaking a large read into multiple smaller calls to SSL_read)? It seems like fixing this at the SSL layer is more appropriate than trying to work around it at the HTTP layer, and thus impacting the performance of all HTTP fetches (whether using TLS or not, and whether >2GB or not).

Bruce, you're discussing the *revert* of the bpo-36050 change as if it was a new change. It isn't, it is literally returning to a state that was previously tested and worked fine since Python 3.5. By reverting, we're choosing correctness over performance here.

Introducing a new implementation for bpo-36050 is now out of scope for 3.9 as it's a performance-oriented change which we cannot accept in a bugfix release.
History
Date User Action Args
2021-07-29 16:08:26lukasz.langasetrecipients: + lukasz.langa, ronaldoussoren, christian.heimes, methane, bmerry, jakirkham, matan1008, amacd31, jan-xyz
2021-07-29 16:08:26lukasz.langasetmessageid: <1627574906.16.0.720768349468.issue42853@roundup.psfhosted.org>
2021-07-29 16:08:26lukasz.langalinkissue42853 messages
2021-07-29 16:08:26lukasz.langacreate