fix(ObjectStorePreviewStorage): Close count stream wrapper

[CarlSchwan]

• Resolves: [Bug]: ObjectStorePreviewStorage returns size 0 with OpenStack Swift, causing “Unable to write preview file” #63801

Summary

So that the size is finalized

TODO

• ...

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)
• Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
• Milestone added for target branch/version (ex: 32.x for stable32)

AI (if applicable)

• The content of this PR was partly or fully generated using AI

[CarlSchwan]

/backport to stable35

[CarlSchwan]

/backport to stable34

[CarlSchwan]

/backport to stable33

[jo23sh]

Production data point in favour of backporting this to stable34.

We hit this on 34.0.2 with OpenStack Swift as primary storage. writePreview() returns $size = 0 on every call because Swift::writeObject() never closes the stream it is handed, so the CountWrapper callback never fires. savePreview() then throws Unable to write preview file even though the upload to Swift succeeded.

The impact is bigger than log noise, because the preview is never recorded in oc_previews:

• No preview has been cached since the 34 upgrade. Last row in oc_previews is 2026-08-10 00:38, hours before we upgraded that morning — 21 days, zero new rows, against 14,781 image files.
• Every thumbnail request regenerates from scratch, indefinitely. ~99% of the failures are GET /core/preview, and a 32×32 request still goes through getMaxPreview(), so each one fully decodes the source and builds a 4096×4096 preview.
• Orphaned objects accumulate. The upload succeeds before the throw, so each attempt leaves an object nothing will ever read: 26,306 objects / 10.5 GB in 21 days, ~34% of all objects in the container.
• On a small (4 GB) instance this exhausted memory and required a hard reset.

For comparison the files path gets this right — ObjectStoreStorage::writeStream() closes the count stream before reading the size:

$this->objectStore->writeObject($urn, $countStream, ...);
if (is_resource($countStream)) {
    fclose($countStream);
}
$stat['size'] = $totalWritten;

Carrying this patch locally meanwhile; it applies cleanly to 34.0.2. Happy to test a stable34 backport.