Changeset 9d3e646 for fedd/federation/remote_service.py

Timestamp:

Dec 3, 2008 9:41:58 AM (15 years ago)

Author:

Ted Faber <faber@…>

Branches:

axis_example, compt_changes, info-ops, master, version-1.30, version-2.00, version-3.01, version-3.02

Children:

375ca67

Parents:

46a0f7a

Message:

Differentiate between a failure to connect to a service and the service being
provided using an unexpected protocol. Deal with the internal connection error
more cleanly when the connection is being made to an internal service. That
is, when an internal service is down, propagate a clean error out to the
caller. Before this change, the internal service failure would look like a
protocol error on the external service (SOAP being provided by XMLRPC, for
example) and try the other protocol, which would fail with a misleading error.

File:

• fedd/federation/remote_service.py (modified) (2 diffs)

fedd/federation/remote_service.py

@@ -346 +346 @@
                     { self.request_body_name: r}, self.encap_fedids))
         except socket_error, e:
-            raise service_error(service_error.protocol, 
+            raise service_error(service_error.connect, 
                     "Cannot connect to %s: %s" % (url, e[1]))
         except SSLError, e:
-            raise service_error(service_error.protocol,
+            raise service_error(service_error.connect,
                     "SSL error contacting %s: %s" % (url, e.message))
         except Fault, f:
@@ -407 +407 @@
             resp = remote_method(msg)
         except socket_error, e:
-            raise service_error(service_error.protocol, 
+            raise service_error(service_error.connect, 
                     "Cannot connect to %s: %s" % (url, e[1]))
         except SSLError, e:
-            raise service_error(service_error.protocol,
+            raise service_error(service_error.connect,
                     "SSL error contacting %s: %s" % (url, e.message))
         except ParseException, e: