That looks good. Perhaps we could use requests.codes.ok instead of 200, but maybe that's just splitting hairs about magical constants... :) On 23/06/17 20:36, Pat Riehecky wrote: > --- > centos.git.repolist.py | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/centos.git.repolist.py b/centos.git.repolist.py > index 75122ae..e135ca7 100755 > --- a/centos.git.repolist.py > +++ b/centos.git.repolist.py > @@ -13,7 +13,7 @@ import requests > import simplejson as json > import sys > > -RPCURL = "https://git.centos.org/rpc?req=LIST_REPOSITORIES" > +RPCURL = "https://git.centos.org/rpc/?req=LIST_REPOSITORIES" > > def read_args(): > ''' > @@ -48,6 +48,11 @@ def get_repo_list(url, branch, projectpath): > print err_msg > sys.exit(1) > > + if req.status_code != 200: > + print "Unable to access gitblit api at " + url > + sys.exit(1) > + > + > payload = req.text > repos = json.loads(payload) > branchname = 'refs/heads/' + branch > @@ -68,6 +73,6 @@ def main(): > repos = get_repo_list(url=options.url, branch=options.branch, projectpath=options.project) > if repos: > print '\n'.join(repos) > - > + > if __name__ == "__main__": > main() >