Original Drupal 7 security patch attached.

CommentFileSizeAuthor
#7 interdiff.txt2.82 KBtim.plunkett
#7 csrf-2141041-7.patch3.18 KBtim.plunkett
#4 interdiff.txt2.11 KBamateescu
#4 2141041-4.patch3.55 KBamateescu
#2 2141041-1.patch3.05 KBamateescu
security-valid-token-82213-14.patch4.37 KBklausi

Comments

amateescu’s picture

amateescu commented
Status: Active » Needs review
Issue tags: +PHPUnit

Here's the forward port. Couldn't find how to make PHPUnit ignore fatal errors yet..

amateescu’s picture

amateescu commented
StatusFileSize
new 2141041-1.patch3.05 KB

aaand the patch.

Status: Needs review » Needs work

The last submitted patch, security-valid-token-82213-14.patch, failed testing.

amateescu’s picture

amateescu commented
Status: Needs work » Needs review
StatusFileSize
new 2141041-4.patch3.55 KB
new interdiff.txt2.11 KB

This should be one way to do it..

The last submitted patch, 4: 2141041-4.patch, failed testing.

amateescu’s picture

amateescu commented

4: 2141041-4.patch queued for re-testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
StatusFileSize
new csrf-2141041-7.patch3.18 KB
new interdiff.txt2.82 KB

Cleaned up the unit test some more, dataProvider++

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

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

Thanks a lot for the fast turnaround on these SA follow-ups!

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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