This is in 6.x, and applies to 7.x.

Patch has some failed hunks.

CommentFileSizeAuthor
#1 SA-319466.patch4.05 KBDave Reid
core7-csrf-cached-forms_0.patch4 KBcatch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs work » Needs review
FileSize
SA-319466.patch4.05 KB

Think I got it. Running all tests in the meantime...

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Reviewed & tested by the community

Meh... form api, blog, and poll tests all passed. Rest of the suite is running.....so......slow, almost half done, but no errors so far.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Reviewed & tested by the community » Needs work

Oh wait... had 1 fail:
File sites/default/files/simpletest/html-1.txt was allowed to be uploaded Other upload.test 121 UploadTestCase->testFilesFilter()

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Oh shoot. ;) So I shouldn't have committed #1, eh? ;)

I've been kind of lax running the tests tonight in the interest of getting caught-up SA-wise, so make sure that this doesn't happen with the patch reverse-applied. Could've been one of the other patches.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs work » Reviewed & tested by the community

You're correct. It still fails without the patch. So looks like this is good to go.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

OK cool. :) Committed to HEAD. Thanks!

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.