Merge lp://staging/~brendan-donegan/checkbox-certification/return_status_url into lp://staging/checkbox-certification

One thing to fix:

25 + logging.info("Submission status: " + base_url + status['url'])

Best to use logging's string interpolation stuff here:

logging.info("Submission status: %s%s", base_url, status['url'])

Also, a couple of comments.

Do you want to handle the situation in which the server doesn't return json (in which case json.loads will throw an exception), and the situation in which the json is *not* what you expect (status['url'] will throw KeyError)?

Then, a minor nitpick:

17 + split_url = args.rest_transport.split('//')
18 + base_url = '//'.join([split_url[0],
19 + split_url[1].split('/')[0]])

This works, but looks a bit crazy, I suggest the following code, which is also only 3 lines (including import) and IMHO clearer in what it does:

import urllib.parse
url_parts = urllib.parse.urlparse(args.rest_transport)
base_url = urllib.parse.urlunparse((url_parts.scheme, url_parts.netloc, "", "", ""))

Your code works fine, however, so I'm cool either way.

Good comments, thanks! I hope the code I've pushed looks better.

Awesome, glad the urllib example was useful. +1, thanks!