I would like there to be a single configuration variable which causes securelogin to secure all forms. I you agree, I would be happy to write the patch.

Comments

dixon_’s picture

dixon_
he/him
Primary language English
Location France
commented
Status: Active » Closed (duplicate)

At some point there was some work on this here: #295069: Support for securing any form. Seem to have staled though...

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Closed (duplicate) » Active

It's not a duplicate. That issue is about securing any form. My feature request is a single checkbox that would secure all forms.

mfb’s picture

mfb
they or he
commented
Status: Active » Fixed

I committed this feature to the dev branch - please test when you have a chance :)

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

The code looks good. I can't test it properly because we have moved to HTTPS-always for all our servers. One suggestion: In securelogin_form_alter(), the two conditions in the if...elseif could be in a single "if" with an "||" operator.

Status: Fixed » Closed (fixed)

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