It would be a nice addition, if the module detects, if the page is accessed via http or https, and redirect to the protocol specified in the module configuration.

CommentFileSizeAuthor
#3 add_check_for_https-2382581-3.patch1.3 KBbahuma20
#1 add_check_for_https-2382581-1.patch1.04 KBbahuma20
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bahuma20’s picture

bahuma20
Primary language German
Location Germany
CreditAttribution: bahuma20 commented
Status: Active » Needs review
FileSize
add_check_for_https-2382581-1.patch1.04 KB

Implemented this. Here's the patch.

bahuma20’s picture

bahuma20
Primary language German
Location Germany
CreditAttribution: bahuma20 commented
bahuma20’s picture

bahuma20
Primary language German
Location Germany
CreditAttribution: bahuma20 commented
FileSize
add_check_for_https-2382581-3.patch1.3 KB

New, working patch.

malcolm_p’s picture

malcolm_p CreditAttribution: malcolm_p commented

I've applied this patch as well, very useful. I'd like to see it in the next release.

timhilliard’s picture

timhilliard CreditAttribution: timhilliard at Acquia commented
Status: Needs review » Closed (won't fix)

I think using something like securepages or a htaccess rule would be better suited to this. The code is also incomplete as it does not take into account reverse proxies which use alternate headers to specify that the request is over HTTPS.