Closed
Bug 634219
Opened 14 years ago
Closed 14 years ago
Wrong "memcache compressed flag" value
Categories
(Cloud Services Graveyard :: Server: Sync, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: znarfor, Assigned: znarfor)
Details
Attachments
(1 file)
2.49 KB,
patch
|
telliott
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b12pre) Gecko/20110211 Firefox/4.0b12pre
Build Identifier:
When using memcache in PHP, the compressed flag should be an integer, not a boolean. In some recent setups, when it's a boolean, PHP issue a warning error, which break the expected JSON output.
This can be easily fixed by using the MEMCACHE_COMPRESSED constant.
It seems this problem only appear with Memcache extension 3.0.3 and upper:
http://pecl.php.net/package-info.php?package=memcache&version=3.0.3
Reproducible: Always
Steps to Reproduce:
1. Install Firefox Sync PHP Server on Ubuntu 10.10 with default packages (php5, php5-memcache)
2. Get URL: http://example.com/firefox-sync/1.0/username/info/collections
Actual Results:
PHP issue a Warning error:
The lowest two bytes of the flags array is reserved for pecl/memcache internal use
Expected Results:
PHP should not issue any warning.
Assignee | ||
Comment 1•14 years ago
|
||
Updated•14 years ago
|
Attachment #512430 -
Flags: review?(telliott)
Updated•14 years ago
|
Assignee: nobody → znarfor
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•14 years ago
|
||
Comment on attachment 512430 [details] [diff] [review]
Patch replacing 'true' boolean with MEMCACHE_COMPRESSED constant
Looks good. I'll check it in
Attachment #512430 -
Flags: review?(telliott) → review+
Comment 3•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•2 years ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•