Closed pwolanin closed 10 years ago
Man, you're quick! I was just about to look into it, myself! :) :+1:
I haven't even gotten the security alert from Drupal.org yet! ;)
Please do some testing if you can - it's possible this change will cause issues for anonymous user forms when the site is behind Varnish. I haven't tagged the release yet.
I just rolled it out to our production box. The AJAX forms look good so far.
On Wed, Apr 16, 2014 at 8:12 PM, pwolanin notifications@github.com wrote:
Please do some testing if you can - it's possible this change will cause issues for anonymous user forms when the site is behind Varnish. I haven't tagged the release yet.
Reply to this email directly or view it on GitHubhttps://github.com/pressflow/6/pull/83#issuecomment-40667639 .
That Chain News form doesn't seem to be built using Drupal's Form API
Hmm ... I have one more, but it's an admin form. I just ran through its functionality and it seems to be working fine.
On Wed, Apr 16, 2014 at 8:22 PM, pwolanin notifications@github.com wrote:
That form doesn't seem to be built using Drupal's Form API
Reply to this email directly or view it on GitHubhttps://github.com/pressflow/6/pull/83#issuecomment-40668209 .
Trying with an anonymous user with a site behind Varnish I'm not having much success - the form build ID is not getting updated as it should.
Hrm ... I use Varnish, but I don't have any public AHAH forms. The admin form seems to be working fine.
On Thu, Apr 17, 2014 at 8:45 AM, pwolanin notifications@github.com wrote:
Trying with an anonymous user with a site behind Varnish I'm not having much success - the form build ID is not getting updated as it should.
Reply to this email directly or view it on GitHubhttps://github.com/pressflow/6/pull/83#issuecomment-40709776 .
fixes #82