As a side-effect of #1728338: Secret is shared amongst varnish3 servers I uploaded a patch to do some general code cleanup.

Direct link: https://drupal.org/files/issues/varnish-multiple_control_keys-1728338-12...

Would be nice to get this in before #2172621: Stable release?

CommentFileSizeAuthor
#1 varnish-multiple_control_keys-1728338-12-cleanup.patch6.4 KBmgifford
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford at OpenConcept Consulting Inc. commented
Version: 7.x-1.x-dev » 7.x-1.0-beta3
Related issues: +#1728338: Secret is shared amongst varnish3 servers
FileSize
varnish-multiple_control_keys-1728338-12-cleanup.patch6.4 KB

Just attaching code to this issue.

manarth’s picture

manarth CreditAttribution: manarth as a volunteer commented
Status: Needs review » Needs work

Patch needs a re-roll, it doesn't apply against 7.x-1.x-dev any more.