Ticket #946 (closed defect: fixed)

Opened 12 years ago

Last modified 11 years ago

The Discovery Web Service port address is hard-coded

Reported by: selatham Owned by: sdonegan
Priority: critical Milestone: NDG2 Cleanup
Component: discovery Version:
Keywords: DiscoveryService Cc:

Description

Now that content in Production and Development are diverging I can see that the Production service is actually using the glue SOAP WS (it's picking up records that only appear there). This seems to be because it is hard-coded in  http://proj.badc.rl.ac.uk/ndg/browser/TI05-delivery/ows_framework/trunk/ows_server/ows_server/models/DiscoveryService_services.py?rev=3249

15 	class DiscoveryServiceLocator:
16 	    DiscoveryServicePortType_address = "http://glue.badc.rl.ac.uk:8080/axis2/services/DiscoveryService"
17 	    def getDiscoveryServicePortTypeAddress(self):
18 	        return DiscoveryServiceLocator.DiscoveryServicePortType_address
19 	    def getDiscoveryServicePortType(self, url=None, **kw):
20 	        return DiscoveryServiceSOAP11BindingSOAP(url or DiscoveryServiceLocator.DiscoveryServicePortType_address, **kw)

Change History

comment:1 Changed 12 years ago by lawrence

Actually, if you look carefully, it's not hardcoded here: it has url or DiscoverySserviePortTypeAddresss? ...

You'll note that the version of ndgSearch in ndgUtils exploits url to avoid this problem, and what we need to do is implement the ndgUtils version in production, and use the configfile to point to the backend. I thought I'd done this, but perhaps the pre-xmas euphoria has addled my memory ...

We do need to sort out branching, because I would like to update to using ndgUtils throughout, which would take some effort ...

comment:2 Changed 12 years ago by selatham

  • Priority changed from blocker to critical

I have greated a LIVE-ONLY version of the ows_server python2.5 egg with the default DiscoveryServicePortType_address= " http://glue.badc.rl.ac.uk:8080/axis2/services/DiscoveryService". This has been built and installed and seems to be working.

Bryan still needs to make a branch using ndgUtils...for the next release to live.

comment:3 Changed 11 years ago by lawrence

  • Keywords DiscoveryService added
  • Component changed from DiscoveryService to discovery

Moved from DiscoveryService? component to discovery as part of NDG2 cleanup

comment:4 Changed 11 years ago by lawrence

  • Owner changed from lawrence to sdonegan

comment:5 Changed 11 years ago by sdonegan

  • Status changed from new to closed
  • Resolution set to fixed

grrr. Now I find it. FIXED!!!

Note: See TracTickets for help on using tickets.