DPS have (for some reason) decided to go and move all their services on to sec.paymentexpress.com and off www.paymentexpress.com

This probably affects more than PxPost, but here's a patch to switch the URL for that module. Someone should also review PxPay, PxAccess, etc too.

Customers need to make this change before the 30th of August. The new URLs are already live and can be used in production.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aaron1234nz’s picture

subscribe

Details of the migration can be found at http://www.paymentexpress.com/migration.html

xurizaemon’s picture

Component: Code - PXPost » Code
FileSize
4.09 KB

Tested uc_dps_pxaccess, testing welcome for others. This patch should update URLs for PXAccess, PXPost and PXPay.

branch in git for this issue

Thanks for the nudge all.

katiebot’s picture

Thanks for the patch Chris! Looks like the URL for PxAccess should be https://sec.paymentexpress.com/pxpay/pxpay.aspx instead - let me know if I should attempt to create a new patch.

xurizaemon’s picture

There were some issues with that patch.

It does appear that the URL for PxAccess is "pxpay.aspx" while the URL for PxPay is "pxaccess.aspx". Weird, but apparently correct. Maybe that all makes sense to DPS.

Will post an update shortly.

xurizaemon’s picture

Here's an updated patch. Please test and review. I've run test transactions against PxPost, PxPay, PxAccess and it seems to behave OK.

Tarball available @ http://drupalcode.org/project/uc_paymentexpress.git/snapshot/refs/heads/...

dollar_dad’s picture

I'm not sure this is working, I'm getting a blank screen when I'm being returned from the DPS form.

The $_REQUEST is not getting the 'result' variable.

xurizaemon’s picture

@dollar_dad: See http://drupal.org/node/158043 for methods of debugging WSOD - and be careful to obfuscate any sensitive data before posting debug data in a public forum.

Were you seeing this issue before applying the patch? (A separate email from your colleague suggested to me that this might be the case.)

dollar_dad’s picture

Yes this error was occurring before I applied the patch but not before the changes by DPS. Just to confirm this isn't the wsod. It's a blank screen.

Currently,
view cart
checkout
review order
submit
(takes you to DPS form)
enter cards details
Submit DPS Form
(DPS Payment Complete Form - response = approved)
Click Next
(you are redirected back to the website with a url of
/uc_dps_pxaccess/complete?result=cb4d6a5e5a94e.......

I've done a few tests inside uc_dps_pxaccess.module file and can see that we get all the way to
line 303 inside the uc_dps_pxaccess_callback_process function.
I've tested that the call
if (isset($_REQUEST["result"]))
which is returning false.
therefore return $output is empty.

I've checked the server and no php errors are being logged

xurizaemon’s picture

OK, @dollar_dad that can go to a separate issue if there's no indication it's affected by the patch(es) filed here.

I've copied your comments to #1262340: $_GET value "result" empty on return from DPS due to suhosin configuration, please use that issue instead.

xurizaemon’s picture

I've committed this to 6.x-1.x-dev.

Review from people using 6.x-1.x-dev in production wanted. Ideally we release the update before DPS actually flip the switch and sites running 6.x-1.0 break.

If you are using 6.x-1.x-dev and this update doesn't break your site, please update this issue and say

"I'm using (uc_dps_pxaccess|uc_dps_pxpay|uc_dps_pxpost) and this works for me."

Thanks.

snlnz’s picture

Hey Chris thanks for the heads up!

For what ever reason I never managed to get uc_paymentexpress to do pxpay. So I still use: uc_dps_pxpay from http://drupal.org/node/653850

When I applied your patch it everything is still working fine on 6.x-1.1.
Test transactions completed. Funny thing is when I checkout, I still end up on DPS's payment page but at URL:
https://www.paymentexpress.com/pxpay/pxpay.aspx
I thought I should be redirected to the new URL:
https://sec.paymentexpress.com/pxpay/pxaccess.aspx

I didn't check this prior to patching but it's not really adding up? :O
Is this something that is happening at DPS or is the URL not applying as configured in the module.

I'm not convinced everything is working as it should if the URLs aren't matching up at checkout time or is this normal behaviour?

xurizaemon’s picture

Status: Needs review » Fixed

There was a typo in the PxAccess .install file which is fixed now. Please upgrade to the next 6.x-1.x-dev tarball (or grab from Git) and test.

Have tested and confirmed this one works ... marking fixed.

For what ever reason I never managed to get uc_paymentexpress to do pxpay. So I still use: uc_dps_pxpay from http://drupal.org/node/653850

That's a much older version of this payment module, and is no longer supported.

Status: Fixed » Closed (fixed)

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