From 0578bdffcb0599d2c4f06998281ad968bec43ac0 Mon Sep 17 00:00:00 2001
From: Timotej Lazar <timotej.lazar@araneo.si>
Date: Tue, 16 Jan 2024 21:51:47 +0100
Subject: [PATCH] Use file-based certificates by default

---
 margfools | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/margfools b/margfools
index 25b34f0..71634ad 100755
--- a/margfools
+++ b/margfools
@@ -40,9 +40,9 @@ def sign(data, key, pin=None, engine=None):
 if __name__ == '__main__':
     parser = argparse.ArgumentParser(description='Fake the MargTools application.')
     parser.add_argument('url', type=urllib.parse.urlparse, help='bc-digsign:// url')
+    parser.add_argument('-e', '--engine', type=str, help='"pkcs11" for smart card')
     parser.add_argument('-k', '--key', type=pathlib.Path, help='key file')
     parser.add_argument('-c', '--cert', type=pathlib.Path, help='certificate file')
-    parser.add_argument('-e', '--engine', type=str, help='"pkcs11" for smart card')
     args = parser.parse_args()
 
     try:
@@ -51,9 +51,11 @@ if __name__ == '__main__':
         url = params['baseUrl'][0]
         token = params['accessToken'][0]
 
-        # if missing, get key and cert from section [url] in ~/.margfools
+        # if missing, get options from section [url] in ~/.margfools
         config = configparser.ConfigParser()
         config.read(os.path.expanduser('~') + '/.margfools')
+        if not args.engine:
+            args.engine = config.get(url, 'engine', fallback=None)
         if not args.key:
             args.key = config.get(url, 'key')
         if not args.cert:
@@ -61,8 +63,6 @@ if __name__ == '__main__':
         if not args.key:
             print('key not specified', file=sys.stderr)
             sys.exit(1)
-        if not args.engine:
-            args.engine = config.get(url, 'engine')
 
         pin = None