-
-
/trunk/reviewboard/contrib/tools/post-review (Diff revision 1) The core of the problem is that the URL needs to have a trailing newline and, in your example, does not. What I'd recommend instead is ensuring that any URL we find (via command line parameter, repository properties, or config file) always ends in a trailing "/".
Fix bug [#796] to allow post-review tool to work even if reviewboard was installed on a non-root url (e.g. http://www.example.com/reviews)
Review Request #689 — Created Jan. 5, 2009 and submitted
This is to fix a bug I found on the post-review tool (issue #796). The fix involves modifying the url returned by _make_url so that the application path (e.g. 'reviews' in the url 'http://www.example.com/reviews') is used to build the correct url given a relative path for the json api calls in the code e.g. '/api/json/repositories' - the original _make_url method would return 'http://www.example.com/api/json/repositories' which would work only if the reviewboard was installed on '/'. The fix now changes _make_url so that it returns the value 'http://www.example.com/reviews/api/json/repositories'