Bugfix: revert /etc/resolv.conf back to the original if it was modifi…

…ed in a finally block (only for connect code invocations), ran tests on macOS

openpyn/openpyn.py

@@ -564,6 +564,10 @@ def run(init: bool, server: str, country_code: str, country: str, area: str, tcp
             return 1
         except SystemExit:
             logger.info("Shutting down safely, please wait until process exits")
+        finally:
+            if skip_dns_patch is False:
+                # Darwin: Revert /etc/resolv.conf back to the original if it was modified
+                subprocess.call(["sudo", "-u", sudo_user, __basefilepath__ + "scripts/revert-manual-dns-patch.sh"])
 
     elif server:
         try:
@@ -629,14 +633,14 @@ def run(init: bool, server: str, country_code: str, country: str, area: str, tcp
             return 1
         except SystemExit:
             logger.info("Shutting down safely, please wait until process exits")
+        finally:
+            if skip_dns_patch is False:
+                # Darwin: Revert /etc/resolv.conf back to the original if it was modified
+                subprocess.call(["sudo", "-u", sudo_user, __basefilepath__ + "scripts/revert-manual-dns-patch.sh"])
 
     else:
         logger.info("To see usage options type: 'openpyn -h' or 'openpyn --help'")
 
-    if skip_dns_patch is False:
-        # Darwin: Revert /etc/resolv.conf back to the original if it was modified
-        subprocess.call(["sudo", "-u", sudo_user, __basefilepath__ + "scripts/revert-manual-dns-patch.sh"])
-
     # if everything went ok
     return 0