aboutsummaryrefslogtreecommitdiffstats
path: root/nextcloud_apps
diff options
context:
space:
mode:
authornico <nico@magicbroccoli.de>2018-10-20 20:47:03 +0200
committernico <nico@magicbroccoli.de>2018-10-20 20:47:03 +0200
commitdeb46a523e11e0c0f37930ad2a65b86f39f7eb8a (patch)
treec9520990d277240ef39c38dc8a6b443f86ea3929 /nextcloud_apps
parentffa77ad80004c24938dab96661d67cb9320a5599 (diff)
further code optimazation
* reduced code clutter - removed for iterations in result and config lists + added list comprehension * updated config
Diffstat (limited to 'nextcloud_apps')
-rwxr-xr-xnextcloud_apps38
1 files changed, 19 insertions, 19 deletions
diff --git a/nextcloud_apps b/nextcloud_apps
index e7a5bbc..14427f8 100755
--- a/nextcloud_apps
+++ b/nextcloud_apps
@@ -49,7 +49,7 @@ class NextcloudApps:
return data
def run(self):
- # init requests session with specific header and credentials
+ # init request session with specific header and credentials
with requests.Session() as s:
# read credentials from env
s.auth = (os.environ.get('username'), os.environ.get('password'))
@@ -60,29 +60,30 @@ class NextcloudApps:
# request the data
r = s.get(os.environ.get('url'))
- # if status code is successful continue
- if r.status_code == 200:
- api_response = r.json()
- result = self.get_data(api_response)
+ # if status code is successful continue
+ if r.status_code == 200:
+ result = self.get_data(r.json())
- for key in result.keys():
- print('\n'.join(result[key]))
+ # for key in results print every entry in dict
+ [print('\n'.join(result[key])) for key in result.keys()]
- elif r.status_code == 996:
- print('server error')
- elif r.status_code == 997:
- print('not authorized')
- elif r.status_code == 998:
- print('not found')
- else:
- print('unknown error')
+ elif r.status_code == 996:
+ print('server error')
+ elif r.status_code == 997:
+ print('not authorized')
+ elif r.status_code == 998:
+ print('not found')
+ else:
+ print('unknown error')
def main(self):
- # check if first argument is config or autoconf if not fetch data
+ # check if any argument is given
if sys.argv.__len__() >= 2:
+ # check if first argument is config or autoconf if not fetch data
if sys.argv[1] == "config":
- for key in self.config().keys():
- print('\n'.join(self.config()[key]))
+ # for key in config().keys() print every entry in dict
+ [print('\n'.join(self.config()[key])) for key in self.config().keys()]
+ # MUNIN_CAP_DIRTYCONFIG capability
if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1':
self.run()
elif sys.argv[1] == 'autoconf':
@@ -96,4 +97,3 @@ class NextcloudApps:
if __name__ == "__main__":
NextcloudApps().main()
- quit(0)