Only apply workaround when 'key_filename' is defined. Closes #5.

jaraco · Aug 6, 2023 · 16cb1ab · 16cb1ab
1 parent 48b8e3c
commit 16cb1ab
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/jaraco/fabric/monkey.py b/jaraco/fabric/monkey.py
@@ -8,11 +8,12 @@ def workaround_2090(func):
 
     @functools.wraps(func)
     def wrapper(c, *args, **kwargs):
-        c.connect_kwargs['key_filename'][:] = [
-            key
-            for key in c.connect_kwargs['key_filename']
-            if __import__('os').path.exists(key)
-        ]
+        if 'key_filename' in c.connect_kwargs:
+            c.connect_kwargs['key_filename'][:] = [
+                key
+                for key in c.connect_kwargs['key_filename']
+                if __import__('os').path.exists(key)
+            ]
         return func(c, *args, **kwargs)
 
     return wrapper
diff --git a/newsfragments/5.bugfix.rst b/newsfragments/5.bugfix.rst
@@ -0,0 +1 @@
+Fixed issue with workaround for fabric/fabric#2090.