CURLOPT_SSL_VERIFYHOST no longer accepts a value of 1 - it needs to be set to 2 (in function _commerce_sagepay_request_post() of commerce_sagepay_common.inc). PHP 5.5. upwards generates a non-critical warning message.

CommentFileSizeAuthor
#1 commerce_sagepay-ssl_verifyhost-2427173-1.patch661 bytesdwkitchen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dwkitchen’s picture

dwkitchen CreditAttribution: dwkitchen for PlanesTV commented
FileSize
commerce_sagepay-ssl_verifyhost-2427173-1.patch661 bytes

Patch Attached

dwkitchen’s picture

dwkitchen CreditAttribution: dwkitchen for PlanesTV commented
Status: Active » Needs review
Chim’s picture

Chim CreditAttribution: Chim commented

Approved
In production environments the value of this option should be kept at 2 (default value).

http://php.net/manual/en/function.curl-setopt.php

Chim’s picture

Chim CreditAttribution: Chim commented
Status: Needs review » Reviewed & tested by the community
richH’s picture

richH CreditAttribution: richH commented

Hi,

can this be put into a release please?

Thanks
Rich

richH’s picture

richH CreditAttribution: richH commented

Hi,

can you please put this into a release?

thanks
Rich

  • 34b4236 committed on 7.x-1.x 
    Issue #2427173 by dwkitchen: PHP 5.5+ curl compatibility

Changes a Curl...
ikos’s picture

ikos CreditAttribution: ikos at Inviqa commented
Status: Reviewed & tested by the community » Fixed

Committed (finally). Thanks for reporting.

richH’s picture

richH CreditAttribution: richH commented

:-)
Thanks for doing that.

Status: Fixed » Closed (fixed)

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