(reported by @dollar_dad in #1187578: DPS changing endpoint URLs)

I'm getting a blank screen when I'm being returned from the DPS form.

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

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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xurizaemon’s picture

Sounds like you may have some setup where $_REQUEST is not populated initially, or perhaps a module which clears the values before this module can get to them.

Would be interested to see phpinfo() on your server if you can flick me a link (email is OK, or post here).

Please try this patch and see if it helps?

dollar_dad’s picture

Added Patch
Made a transaction
error log message
Invalid $_GET data:

Array
(
[q] => uc_dps_pxaccess/complete
[userid] => DevolLtd_Dev
)

dollar_dad’s picture

Title: $_REQUEST['result'] empty on return from DPS » SOLVED

Just to update this thread

It appears the changes to DPS return on order complete send a URL link that is nearly 3000 characters long.
This issue is being caused by the Suhosin configuration, which has (by default setting) suhosin.get.max_value_length=512 on our server.

Once we changed this the patched version worked fine.

A big thanks goes to grobot for solving this issue.

xurizaemon’s picture

Title: SOLVED » $_GET value "result" empty on return from DPS due to suhosin configuration
Category: bug » support
Status: Needs work » Fixed

Thanks

Status: Fixed » Closed (fixed)

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

xurizaemon’s picture

OK, there seem to be enough people askign about this one that we need to handle it more obviously.

Can't find any good examples in core or contrib where we have to futz around with apache extension settings, but this is triggered by default config on Debian (at least) so worth handling.

This is ugly. I hate it.

NOT THIS PATCH.

xurizaemon’s picture

NOT THIS ONE EITHER.

xurizaemon’s picture

Status: Closed (fixed) » Needs review
FileSize
1.84 KB

Actually, try this patch. I don't want to check suhosin config, just that the result variable has been omitted. It might be some other bung server configuration.

snlnz’s picture

Patch #8 displays a helpful error message on redirect from DPS stating:
Server configuration prevented DPS transaction completion. Please advise site administrator to check Drupal logs for details.

xurizaemon’s picture

Status: Needs review » Fixed

Committed patch from #8. Thanks for testing Karu.

Status: Fixed » Closed (fixed)

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