http://acquia.com/node/393891

This is really bad - login *has* to be done over https:// because the users password is being transmitted. But browsers complain that the CAPATCHA is pulled in over http:// only so throws an alert!! Doesn't look terribly professional. This is a showstopper for me. Please fix this asap!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Title: CAPATCHA ssl/https issue » Use https for CAPTCHAs
Version: 6.x-1.9 » 6.x-1.x-dev
Component: Miscellaneous » Code
Priority: Critical » Normal
Status: Active » Postponed

As it says on http://mollom.com/faq/does-mollom-support-ssl, this could be added eventually. I don't have any control over the Mollom servers, just the Drupal module. Marking as postponed for now.

jasonabc’s picture

thanks Dave - yeah good point - sorry for the rant in the wrong place! Hopefully someone over there will get this sorted.

thanks!

Jason

jasonabc’s picture

just a footnote to this - I also noticed that Internet Explorer 8 goes one further and pops open a dialog box telling the user some content on the page is not being transmitted securely and asks if they only want to see the secure content. If they click "yes" - the CAPTCHA image is not displayed meaning customers are unable to create accounts... Have emailed them (again) so hopefully this will be resolved/fixed soon.

cheers

J

vitis’s picture

Same problem.

I have an https site. The link on the mollom capcha picture is to http. This is always going to give an "insecure..." popup on internet explorer.

I went to where the mollom picture was pointing, http://mollom.com/, and typed in https://mollom.com/ - there was no page there. I thought I'd give that a shot, because that's how I solved a similar problem with a paypal button.

I hate to do it, but I'm going to disable mollom.

sun’s picture

Title: Use https for CAPTCHAs » Use SSL for CAPTCHAs
Assigned: Unassigned » sun
Status: Postponed » Needs review
FileSize
1.36 KB

Attached patch directs mollom.com to respond with https-URLs. I was not able to test this, because testing keys do not seem to have SSL support.

Dries’s picture

This looks good. We'll be able to test it as soon as we upgrade the backend.

sun’s picture

As discussed, only low-level tests for now.

Dries’s picture

Status: Needs review » Fixed

Committed to DRUPAL-6--1 and CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

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

jasonabc’s picture

Status: Closed (fixed) » Active

The free version of Mollom says "No Secure communication (SSL)". So the patches and commits above won't solve this issue for free users - correct? You have to upgrade to the paid service? This is ridiculous. Since CAPTCHAs largely sit on forms that are always protected by SSL due to password and other sensitive data being transmitted, Mollom's free service rather a waste of time/unusable.

sun’s picture

Status: Active » Closed (fixed)

Yes, SSL support is bound to a paid Mollom subscription, and is not supported by Mollom Free.

The reason for that is mostly technical, but relatively easy to get behind:

Every SSL connection requires up to 4x times more processing resources - and thus, hardware resources - compared to a non-SSL connection. Now, if you consider that the Mollom API processes multiple millions of requests every single day, this has a significant impact on the resources. Namely, if every request would run over SSL, only 25% of the Mollom Free subscriptions could be served for free.

Therefore, I can totally get behind Mollom's decision, which essentially says: If you really need a secure connection, then you probably shouldn't use a free subscription.

Or in other words: If SSL isn't supported for free, then Mollom is able to free up 4x times more low-volume sites from spam.

Makes sense? :)

  • Commit ca78cfa on master, fai6, 8.x-2.x, fbajs, actions by Dries:
    - Patch #556162 by sun: allow SSL to be used for CAPTCHAs.
    
    

  • Commit ca78cfa on master, fai6, 8.x-2.x, fbajs, actions by Dries:
    - Patch #556162 by sun: allow SSL to be used for CAPTCHAs.