php / php/doc-extensions

function.wincache-ucache-set fails to store array

Open
#27 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
XML
Stars
2
Forks
6
Avg merge
1d 14h
Merged PRs (30d)
5

Description

From manual page: https://www.php.net/manual/en/function.wincache-ucache-set.php

I've just spent a few hours trying to debug why an IIS server with wincache is hanging, and it all came down to 1 single line

This is a test line that can hang my Wincache / PHP 8.4 / IIS setup

$test_array = ['simple_string'];
$result = wincache_ucache_set('pjs_test_queue', $test_array, $expiration);

This is an array. I have many thousands of non-array items in there and Wordpress strings/articles etc., all in the same instance. No issues at all for anything else. But, add that array, and things just halt. The call never returns. This was a simple test array I did to attempt to replicate the issue. My array was only a few elements, but slightly more complicated. If you change the array to a string in the same code for sanity check, it works fine.

Now, the docs all say it is supposed to support this and internally serialise, but, something seems wrong with my builds, or the docs are not correct?

The fix? If you manually serialize the array FIRST, so

$result = wincache_ucache_set('pjs_test_queue', serialize($test_array), $expiration);

It'll work just fine.

So, I now have a work around and I will not trust the internal serialization that is supposed to happen, but I am reporting here in case it is a general bug that needs to be fixed. Obviously, very, very simple to replicate and test. 2 lines of code.


Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the wincache_ucache_set manual page and the PHP 8.4/IIS reproduction shown in the issue. Verify whether array serialization is documented correctly, then determine whether the appropriate outcome is a documentation correction or escalation to the wincache implementation; the issue is done when that ownership and required change are clear.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.