diff options
author | Robin H. Johnson <robbat2@gentoo.org> | 2007-12-15 05:05:30 -0800 |
---|---|---|
committer | Robin H. Johnson <robbat2@gentoo.org> | 2007-12-18 01:12:21 -0800 |
commit | 807eb03f7cc6a868086ae6580d72dfa3519f2b6e (patch) | |
tree | 79d6601c46b84b9fd567c317fad98d792fe4a975 /gitosis/gitdaemon.py | |
parent | Add new utility function to get a value from a ConfigParser, with a default i... (diff) | |
download | gitosis-dakkar-807eb03f7cc6a868086ae6580d72dfa3519f2b6e.tar.gz gitosis-dakkar-807eb03f7cc6a868086ae6580d72dfa3519f2b6e.tar.bz2 gitosis-dakkar-807eb03f7cc6a868086ae6580d72dfa3519f2b6e.zip |
Use the new getboolean_default utility function to refactor code.
Diffstat (limited to 'gitosis/gitdaemon.py')
-rw-r--r-- | gitosis/gitdaemon.py | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/gitosis/gitdaemon.py b/gitosis/gitdaemon.py index d0f9a7c..86c8a1a 100644 --- a/gitosis/gitdaemon.py +++ b/gitosis/gitdaemon.py @@ -15,6 +15,7 @@ from ConfigParser import NoSectionError, NoOptionError log = logging.getLogger('gitosis.gitdaemon') from gitosis import util +from gitosis.configutil import getboolean_default def export_ok_path(repopath): """ @@ -54,26 +55,20 @@ def _is_global_repo_export_ok(config): """ Does the global Gitosis configuration allow daemon exporting? """ - try: - global_enable = config.getboolean('gitosis', 'daemon') - except (NoSectionError, NoOptionError): - global_enable = False + global_enable = getboolean_default(config, 'gitosis', 'daemon', False) log.debug( 'Global default is %r', {True: 'allow', False: 'deny'}.get(global_enable), ) return global_enable -def _is_repo_export_ok(global_enable, config, name): +def _is_repo_export_ok(global_enable, config, reponame): """ Does the Gitosis configuration for the named reposistory allow daemon exporting? """ - try: - enable = config.getboolean('repo %s' % name, 'daemon') - except (NoSectionError, NoOptionError): - enable = global_enable - return enable + section = 'repo %s' % reponame + return getboolean_default(config, section, 'daemon', global_enable) def _set_export_ok_single(enable, name, dirpath, repo): """ |