RF: avoid code duplication, use os.path.join over all components of t…

…he path
NDAR · Sep 20, 2019 · 85390c3 · 85390c3
1 parent 67b7250
commit 85390c3
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/NDATools/clientscripts/vtcmd.py b/NDATools/clientscripts/vtcmd.py
@@ -100,8 +100,9 @@ def configure(args):
     # create a new config file in user's home directory if one does not exist
 
     NDATools.Utils.logging.getLogger().setLevel(logging.DEBUG)
-    if os.path.isfile(os.path.join(os.path.expanduser('~'), '.NDATools/settings.cfg')):
-        config = ClientConfiguration(os.path.join(os.path.expanduser('~'), '.NDATools/settings.cfg'), args.username,
+    config_file = os.path.join(os.path.expanduser('~'), '.NDATools', 'settings.cfg')
+    if os.path.isfile(config_file):
+        config = ClientConfiguration(config_file, args.username,
                                      args.password, args.accessKey, args.secretKey)
     else:
         config = ClientConfiguration('clientscripts/config/settings.cfg', args.username, args.password, args.accessKey,