pngout generates .tmp.png files, but doesnt remove them

Must be a glitch in the temp file naming scheme, that should not be happening. I’ll have to take a look later today.

Sounds like the permissions are not correct on new files being generated. Which is weird, but it happens sometimes. Just to confirm, are you using BOTH optipng and pngout?

Yes, when the issue happened, i had all tools enabled, including optipng and pngout. upon disabling pngout, i had no longer one of these tmp.png files spawning.

Just to make sure its pngouts fault, I tested it again with just pngout running (no optipng) and it did spawn and keep the additional tmp.png files, even after they were all done.

I did some tests and noticed, that it does not always happen for all .png files within one folder, just some. I suspect that pngout crashes on the files for some reason and fails to remove the erroneous file afterwards?

Here is a copy of a file that does spawn a .tmp.png child when ran though pngout: https://a.loveisover.me/mquubm.png
Also, here is the .tmp.png copy of said file. Notice that the new .tmp.png file is smaller than the original: https://a.loveisover.me/ffnbmw.png

Also, the permissions for the folders are 770 www-data:www-data and for the files, it is 660 www-data:www-data.

I’m unable to replicate it, and both files come back with “no savings” on my dev site. What operating system are you running on the server?

What version of pngout are you running (check the Plugin Status area on the EWWW IO settings page)?

And can you confirm that these are .tmp.png files, and not just a .tmp appended?

Ok, been digging into this more since I wasn’t getting ANY savings reported with just pngout, and it seems pngout does not like to operate on files that end in .tmp, so it appends a .png, which orphans the file.

For some reason, I wasn’t seeing it earlier, but now I can see a whole pile of those files in my uploads folder.

Great! You can replicate it, though why did you flag it as [resolved], as it clearly isn’t fixed yet?

Because I’ve fixed it in dev, and the release is imminent.

Oh alright! thanks a bunch!

just tested the update and it works again. no more orphaned files. thanks a bunch!