This patch uses drupal_http_request() instead of code to open raw socket descriptors.

Also, I took off the code that was modifying some of the POST vars from the IPN server, it would cause IPN to fail. The docs state that the variables must be transmitted back EXACTLY as they are received. Also, I appended the cmd=_notify-validate variable pair instead of prepending it, again per the docs.

CommentFileSizeAuthor
uc_paypal.ipn_.patch9.72 KBcha0s

Comments

davegan’s picture

davegan commented

so there are no functional changes, just code cleanup in this patch, correct?

rszrama’s picture

rszrama commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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