Original code generated url's using a harcoded protocol and took the

hostname from the hostname used to access the tracker. This patch uses
the canonical path from config.ini for generating all links.

Author: rouilj

CHANGES.txt

@@ -43,6 +43,10 @@ Features:
   other than GET that doesn't have a payload. E.G. DELETE, PATCH or
   OPTIONS. Verbs like PUT and POST usually have payloads, so this
   patch doesn't touch processing of these methods. (John Rouillard)
+- Patches to new rest code:
+   - Generated links in responses should use the base url specified
+     in config.ini.
+  (John Rouillard)
 - issue2550833: the export_csv web action now returns labels/names
   rather than id's. Replace calls to export_csv with the export_csv_id
   action to return the same data as the old export_csv action. (Tom

roundup/rest.py

@@ -249,10 +249,8 @@ def __init__(self, client, db):
         # restore actions.
         self.actions = dict (retire = actions.Retire, restore = actions.Restore)
 
-        protocol = 'http'
-        host = self.client.env['HTTP_HOST']
-        tracker = self.client.env['TRACKER_NAME']
-        self.base_path = '%s://%s/%s/rest' % (protocol, host, tracker)
+        # note TRACKER_WEB ends in a /
+        self.base_path = '%srest' % (self.db.config.TRACKER_WEB)
         self.data_path = self.base_path + '/data'
 
     def props_from_args(self, cl, args, itemid=None):