May be related to #556162: Use SSL for CAPTCHAs, but this is a fairly old issue.

My entire site runs under SSL, I'm using Mollom for captchas on webform nodes but browsers (in particular Internet Explorer) give warnings about insecure content on the page, IE being the worst with a popup warning which is less than ideal for our users. It seems like the problem comes from the image that is used in the Mollom captcha uses an http:// url, attempting to change this to https gives a 403 though.

Using the latest Mollom 7.x-2.x-dev branch.

Files: 
CommentFileSizeAuthor
#2 1707154-insecure-content-text-analysis.patch1011 bytesacbramley
PASSED: [[SimpleTest]]: [MySQL] 4,371 pass(es).
[ View ]

Comments

acbramley’s picture

Note: this also happens with text analysis as the disclaimer outputs a link to http://mollom.com/web-service-privacy-policy, this needs to be changed to https://mollom.com/web-service-privacy-policy which is valid.

acbramley’s picture

StatusFileSize
new1011 bytes
PASSED: [[SimpleTest]]: [MySQL] 4,371 pass(es).
[ View ]

Fixes warning for insecure content by outputting privacy policy link based on what protocol the page is using

sun’s picture

Status:Active» Needs review

Nice catch! Let's see whether this patch passes.

Note: On the image CAPTCHA URL, you will automatically get correct SSL URLs with a Plus (or higher) Mollom subscription. SSL is not supported for Mollom Free subscriptions.

sun’s picture

Status:Needs review» Fixed

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

acbramley’s picture

Cool cheers for that :)

Status:Fixed» Closed (fixed)

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

  • Commit 2252e35 on 7.x-2.x, 8.x-2.x, fbajs, actions authored by acbramley, committed by sun:
    Issue #1707154 by acbramley: Fixes insecure content warning for text...

  • Commit 2252e35 on 7.x-2.x, 8.x-2.x, fbajs, actions authored by acbramley, committed by sun:
    Issue #1707154 by acbramley: Fixes insecure content warning for text...