[PATCH] vmod_digest

Nils Goroll slink at schokola.de
Thu Aug 18 16:05:32 CEST 2011


Hi Laurence and all,

sorry for the _very_ late response, I've been highly customer-demand driven and
(and chased by deadlines) during the last months, so much work I started on
Varnish stayed unfinished. The good news is that customers now demand more
Varnish work again, so I hope to return to the more active circle again this year.

>> phk had spotted a race condition in base64_init which I had overlooked (and even
>> commented on sarcastically), so I've added code to pre-generate the base64
>> lookup tables.
>>
>> All other changes are due to the changes to varnish code since then.
> 
> This looks really interesting. Do you have an up to date version of
> this patch? It no longer applies and I was unable to find the base git
> revision in the repository (maybe my lack of git-fu.)

phk wasn't happy with the patch at the time for two reasons, IIUC:

- he didn't like the generator for the base64_tables.h to be done in python
  (which I didn't quite understand at the time because he uses python scripts
   to generate such things all over the place)

- he had asked me to clean up the digest functions code to use
  algorithm-specific function pointers rather than the case statements

Because, at the time, I had no resources to finish these two things, the module
did not make it any official repo at the time.

I am using the code in production, but not on current versions of varnish.

I will definitely need a Varnish 3-compatible digest module by the end of 2011,
so I will come back to this. If anyone else wants to finish this, feel free.

Nils




More information about the varnish-dev mailing list