------------------------------------------------------------
revno: 6728
committer: Barry Warsaw <[email protected]>
branch nick: 3.0
timestamp: Mon 2009-05-11 23:00:30 -0400
message:
A little bit of cleanup.
modified:
src/mailman/rest/testing/server.py
src/mailman/rest/urls.py
=== modified file 'src/mailman/rest/testing/server.py'
--- src/mailman/rest/testing/server.py 2009-05-04 15:33:10 +0000
+++ src/mailman/rest/testing/server.py 2009-05-12 03:00:30 +0000
@@ -52,8 +52,7 @@
def stop(self):
"""Stop the server by firing the event."""
self.event.set()
- # Fire off one more request so the handle_request() will exit. XXX
- # Should we set a .timeout on the server instead?
+ # Fire off one more request so the handle_request() will exit.
fp = urlopen('http://localhost:8001/3.0/system')
fp.close()
self.thread.join()
=== modified file 'src/mailman/rest/urls.py'
--- src/mailman/rest/urls.py 2009-05-10 20:13:02 +0000
+++ src/mailman/rest/urls.py 2009-05-12 03:00:30 +0000
@@ -45,22 +45,18 @@
"""Initialize with respect to a context and request."""
self.context = context
self.request = request
- # XXX Is this strictly necessary?
self.webservice_config = AdminWebServiceConfiguration()
self.version = self.webservice_config.service_version_uri_prefix
self.schema = ('https' if self.webservice_config.use_https else 'http')
self.hostname = config.webservice.hostname
self.port = int(config.webservice.port)
- def __str__(self):
+ def __call__(self):
"""Return the semi-hard-coded URL to the service root."""
path = self._lookup(self.context)
return '{0.schema}://{0.hostname}:{0.port}/{0.version}/{1}'.format(
self, path)
- # XXX Is this strictly necessary?
- __call__ = __str__
-
def _lookup(self, ob):
"""Return the path component for the object.
--
Primary development focus
https://code.launchpad.net/~mailman-coders/mailman/3.0
Your team Mailman Checkins is subscribed to branch lp:mailman.
To unsubscribe from this branch go to
https://code.launchpad.net/~mailman-coders/mailman/3.0/+edit-subscription.
_______________________________________________
Mailman-checkins mailing list
[email protected]
Unsubscribe:
http://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org
