This is spin-off of #3155929: Http Fetcher receiving 304s Do not clear out their queue items.. In comment 2, I describe:

One more issue I've discovered around this, this fetch method starts be creating the temporary file with FileSystemInterface::tempnam(), but if it hits that 304 and throws that excepton, it never cleans up the file. This might not be problem in many systems, but in a load balanced environment, that might be using a non-standard temporary directory, this becomes a big problem as there are no ways to clean this up unless you have devised something separate to handle this. I've mentioned this problem before in another issue #2912130: Missing temporary files in load balanced environments.

CommentFileSizeAuthor
#2 feeds-unlink-empty-tmp-files-3157339-2.patch560 bytesbburg

Comments

bburg created an issue. See original summary.

bburg’s picture

Status: Active » Needs review
StatusFileSize
new560 bytes

Patch.

Status: Needs review » Needs work

The last submitted patch, 2: feeds-unlink-empty-tmp-files-3157339-2.patch, failed testing. View results

jcnventura’s picture

Status: Needs work » Closed (duplicate)

Issue #3080098: Delete Empty Files Created If Empty/Error Exception is Returned is older and has a better patch that the one in #2.