Improved memcached module

[Igor Sysoev]

On Fri, Aug 21, 2009 at 12:47:24PM +0400, Maxim Dounin wrote:

> Hello!
> 
> On Fri, Aug 21, 2009 at 10:06:00AM +0200, Spil Games wrote:
> 
> > Hi,
> > 
> > Attached is a patch which allows both custom headers and custom response
> > codes to be stored in memcached and returned by the memcached module.
> > The Nginx 0.6 version of this patch has been in production on a heavily
> > loaded site (100M unique visitors per month) for over a year with no
> > problems whatsoever. The original thread on this can be found here:
> > 
> > http://www.ruby-forum.com/topic/160379#724717
> > 
> > I think it would be a valuable addition to Nginx to include this in the
> > main distribution, but I'll leave this up to Igor. The patch may be a
> > bit too hacky for his taste.
> > 
> > Feedback appreciated!
> > 
> > Attachments:
> > http://www.ruby-forum.com/attachment/3956/nginx-0.7.61-memcache.final.patch
> 
> There are at least 2 problems with this patch I see:
> 
> 1. It relies on 'HTTP/' prefix on to find out if key in question 
> has headers.  This is bad, as there is no way to return text 
> starting from this string.  IMHO it should be changed to use flags 
> instead.

I agree, it should use a flag and the flag should not conflict with two
perl Cache-Memcached module's flags:
    F_STORABLE      0x0001
    F_COMPRESS      0x0002


-- 
Igor Sysoev
http://sysoev.ru/en/