View Issue Details

IDProjectCategoryView StatusLast Update
0004751Composrcorepublic2021-12-30 21:07
ReporterChris GrahamAssigned ToChris Graham 
SeverityMajor-bug 
Status resolvedResolutionfixed 
Product Version10.0.39 
Fixed in Version10.0.40 
Summary0004751: Chunked transfers that have no HTTP payload will stall
DescriptionIf an HTTP connection results in a chunked transfer, but that chunked transfer is immediately terminated with no data, the downloader will get stuck in an infinite loop (eventually causing a PHP timeout).

This patch treats all the edge cases for chunked downloading MUCH more carefully, and comments the code better.
TagsNo tags attached.
Time estimation (hours)
Sponsorship open

Activities

admin

2021-12-30 21:06

administrator   ~0007234

A hotfix (a TAR of files to upload) has been uploaded to this issue. These files are made to the latest intra-version state (i.e. may roll in earlier fixes too if made to the same files) - so only upload files newer than what you have already. If there are files in a hot-fix that you don't have then they probably relate to addons that you don't have installed and should be skipped. Always take backups of files you are replacing or keep a copy of the manual installer for your version, and only apply fixes you need. These hotfixes are not necessarily reliable or well supported. Not sure how to extract TAR files to your Windows computer? Try 7-zip (http://www.7-zip.org/).

hotfix-4751, 2021-12-30 9pm.tar (115,200 bytes)

Chris Graham

2021-12-30 21:07

administrator   ~0007235

Fixed in https://gitlab.com/composr-foundation/composr/commit/f2abc5750

This specifically was caught because a spammer posted a load of Google links, and Google was rate-limiting us doing link checking. It's not common to be rate limited with an empty HTTP body but chunked encoding turned on, but if it happened it was very nasty.

Issue History

Date Modified Username Field Change
2021-12-30 21:07 Chris Graham Note Added: 0007235