During the testing of the module I found it very difficult to debug problems which were happening with requests and responses, leading me to resort to hacking the code to see what was happening. As these problems will be quite common during installation I think that it is necessary to watchdog and problems which arise.

CommentFileSizeAuthor
#1 commerce_tnt_request_debugging_2226067_1.patch5.02 KBmurrayw

Comments

murrayw’s picture

murrayw commented
StatusFileSize
new commerce_tnt_request_debugging_2226067_1.patch5.02 KB

Here is a patch to do the debugging to watchdog. It does the following:

  • Logs errors for empty requests, empty responses, responses with errors.
  • The full request is logged to watchdog where it exists. The password is removed for security.
  • UPS typo changed to TNT.
  • Slight rejigging of logic with nesting of if statements.
  • A !isset was changed to isset for testing for errors.
  • No errors logged to screen.

  • Commit d9c2aeb on 7.x-1.x: 
    Applied patch from murrayw for Issue #2226067
jbloomfield’s picture

jbloomfield commented
Status: Active » Closed (fixed)