Bug 779700 (SOA-2064) - SOAPProxy Contains Major Logic in Constructor
Summary: SOAPProxy Contains Major Logic in Constructor
Keywords:
Status: CLOSED UPSTREAM
Alias: SOA-2064
Product: JBoss Enterprise SOA Platform 5
Classification: JBoss
Component: JBossESB
Version: 5.0.0 GA
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: FUTURE
Assignee: Ken Johnson
QA Contact:
URL: http://jira.jboss.org/jira/browse/SOA...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-05-03 16:10 UTC by Brad Davis
Modified: 2025-02-10 03:14 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2025-02-10 03:14:42 UTC
Type: Feature Request
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker SOA-2064 0 Minor Closed SOAPProxy Contains Major Logic in Constructor 2012-09-11 11:02:56 UTC

Description Brad Davis 2010-05-03 16:10:48 UTC
Date of First Response: 2010-05-04 03:23:42
Help Desk Ticket Reference: https://enterprise.redhat.com/issue-tracker/832723
project_key: SOA

org.jboss.soa.esb.actions.soap.proxy.SOAPProxy performs major setup logic in constructor.  This keeps us from being able to extend the class to add more features, as the call to the super(ConfigTree ) constructor must be the first line of our constructor. 

We want to be able to add some simple logic to load the URL from a system property, to enable use to move between environments without impact of the codebase.

My suggestion is to move all major setup logic into the initialize method.

Comment 1 Kevin Conner 2010-05-04 07:23:42 UTC
At the moment you should be able to achieve this through the help of static methods.

Comment 2 Kevin Conner 2010-05-04 08:19:35 UTC
One more comment, why are you not using properties within your jboss-esb.xml file???

This should already enable the functionality you are trying to achieve.

Comment 3 Brad Davis 2010-05-04 12:19:47 UTC
Kevin, what does this mean:
At the moment you should be able to achieve this through the help of static methods. 

I have tried the properties within the jboss-esb.xml the way you suggested by email, and it works, so I dropped that out of my codebase.  However, I did not attempt this in my code until I went through JBoss Support, which said it was not possible:  See: https://support.redhat.com/jbossnetwork/restricted/caseDetail.html?caseId=832723  As far as properties in the ESB file, can you point me to the point in the documentation where that is noted?

Comment 4 Brad Davis 2010-05-04 12:27:25 UTC
As far as this bug goes, the logic being in the constructor also leads to the issue of the whole service failing when the proxied web service is temporarily unavailable on server restart.  You suggested: "It is currently a requirement that the wsdl exist when starting the service, although it should be possible to handle this through a barrier of sorts.  What is your scenario?"

The scenario is that, if SOA-P is bounced, and a webservice is unavailable that it should be proxying, it leads to an exception on deployment of the ESB service, and *no way* to recover the service, even if the remote service comes back online.  If this is a requirement of the SoapProxy, it is a design flaw.  The SoapProxy should return a SoapFault until the point that the service becomes available.

Comment 5 Kevin Conner 2010-05-04 12:27:49 UTC
Sorry, you need to continue this through support or take it to the forums.  I have already followed up on the support case but will do so again.

Comment 7 Anne-Louise Tangring 2010-09-21 20:47:01 UTC
This has been triaged for SOA 5.1.0. Not in SOA 5.1.0

Comment 8 Colin Mondesir 2010-10-05 13:04:34 UTC
new related salesforce ticket: https://na7.salesforce.com/500A00000043OkA

Comment 9 Darran Lofthouse 2010-10-27 14:22:35 UTC
Link: Added: This issue is related to PRODMGT-16


Comment 10 Red Hat Bugzilla 2025-02-10 03:14:42 UTC
This product has been discontinued or is no longer tracked in Red Hat Bugzilla.


Note You need to log in before you can comment on or make changes to this bug.