(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.
Comments
Comment #1
xurizaemonSounds 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?
Comment #2
dollar_dad CreditAttribution: dollar_dad commentedAdded Patch
Made a transaction
error log message
Invalid $_GET data:
Array
(
[q] => uc_dps_pxaccess/complete
[userid] => DevolLtd_Dev
)
Comment #3
dollar_dad CreditAttribution: dollar_dad commentedJust 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.
Comment #4
xurizaemonThanks
Comment #6
xurizaemonOK, 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.
Comment #7
xurizaemonNOT THIS ONE EITHER.
Comment #8
xurizaemonActually, 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.
Comment #9
snlnz CreditAttribution: snlnz commentedPatch #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.
Comment #10
xurizaemonCommitted patch from #8. Thanks for testing Karu.