Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-9289

SOAPEventHandler only outputs error to log when secureSoapAnswer = false

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Implemented
    • Affects Version/s: None
    • Fix Version/s: Upcoming Release, 16.11.02
    • Component/s: framework
    • Labels:
      None

      Description

      Commit

      Revision: 1088047
      Author: jleroux
      Date: 02 April 2011 16:23:40
      Message:
      Completes OFBIZ-4218

      Keeps the secure idea but enhance when working in a secure environment (VPN, local network, etc.) to return a comprehensive response. Depends on the secureSoapAnswer property in service.properties, true by default


      Modified : /ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/SOAPEventHandler.java
      Modified : /ofbiz/trunk/framework/service/config/service.properties

      Changes SOAPEventHandler to secure the response on error but it also hides error message from the log if secureSoapAnswer = true (which is the default).

        Issue Links

          Activity

          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Hi Gareth,

          Have you a patch on trunk HEAD?

          Show
          jacques.le.roux Jacques Le Roux added a comment - Hi Gareth, Have you a patch on trunk HEAD?
          Hide
          gareth.carter Gareth Carter added a comment -

          Here is the proposed patch, moving the debug line

          Show
          gareth.carter Gareth Carter added a comment - Here is the proposed patch, moving the debug line
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Thanks Gareth,

          I agree there is no risk showing exception information in log.

          Your patch is in
          trunk r1791288
          R16.11 r1791289

          I decided it was worth and no risk backporting in R16.11

          Show
          jacques.le.roux Jacques Le Roux added a comment - Thanks Gareth, I agree there is no risk showing exception information in log. Your patch is in trunk r1791288 R16.11 r1791289 I decided it was worth and no risk backporting in R16.11

            People

            • Assignee:
              jacques.le.roux Jacques Le Roux
              Reporter:
              gareth.carter Gareth Carter
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development