This need to be done since Drupal 7.

CommentFileSizeAuthor
#14 2486113-14.patch2.45 KBnaveenvalecha
#10 captcha-mda-to-sha256-2486113-2.patch2.25 KBkeshavv
#3 captcha-mda-to-sha256-2486113-1.patch1.92 KBm1r1k
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

undertext’s picture

undertext CreditAttribution: undertext commented
Assigned: Unassigned » undertext
elachlan’s picture

elachlan CreditAttribution: elachlan commented

Hi undertext, How are you progressing on this?

m1r1k’s picture

m1r1k CreditAttribution: m1r1k at Propeople (now part of FFW) for Propeople (now part of FFW) commented
Assigned: undertext » m1r1k
Status: Active » Needs review
FileSize
captcha-mda-to-sha256-2486113-1.patch1.92 KB
hass’s picture

hass CreditAttribution: hass commented
Issue tags: +Needs backport to D7
hass’s picture

hass CreditAttribution: hass commented
Status: Needs review » Needs work

I guess this need to be Crypt::hashBase64 or Crypt::hmacBase64. If the values are used in the form this need to be clean I guess.

The last submitted patch, 3: captcha-mda-to-sha256-2486113-1.patch, failed testing.

Status: Needs work » Needs review

isntall queued 3: captcha-mda-to-sha256-2486113-1.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 3: captcha-mda-to-sha256-2486113-1.patch, failed testing.

ddrozdik’s picture

ddrozdik CreditAttribution: ddrozdik as a volunteer commented
Assigned: m1r1k » ddrozdik

I will update the patch after finishing with #2501699: Fix Image CAPTCHA settings form

keshavv’s picture

keshavv CreditAttribution: keshavv commented
FileSize
captcha-mda-to-sha256-2486113-2.patch2.25 KB

Replaced md5 with sha256 hashes

amit.drupal’s picture

amit.drupal CreditAttribution: amit.drupal at gai Technologies Pvt Ltd commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 10: captcha-mda-to-sha256-2486113-2.patch, failed testing.

elachlan’s picture

elachlan CreditAttribution: elachlan commented

Can you re-roll the patch please?

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Needs work » Needs review
FileSize
2486113-14.patch2.45 KB

Here we go with fresh patch

elachlan’s picture

elachlan CreditAttribution: elachlan commented
Status: Needs review » Fixed
keshavv’s picture

keshavv CreditAttribution: keshavv as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented

Status: Fixed » Closed (fixed)

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