Message398494
> 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. |
|
Date |
User |
Action |
Args |
2021-07-29 16:08:26 | lukasz.langa | set | recipients:
+ lukasz.langa, ronaldoussoren, christian.heimes, methane, bmerry, jakirkham, matan1008, amacd31, jan-xyz |
2021-07-29 16:08:26 | lukasz.langa | set | messageid: <1627574906.16.0.720768349468.issue42853@roundup.psfhosted.org> |
2021-07-29 16:08:26 | lukasz.langa | link | issue42853 messages |
2021-07-29 16:08:26 | lukasz.langa | create | |
|