Merge pull request #7 from lettdigital/fix-signin-envvar

passing os environment variables to signin subprocess to get OP_DEVICE
lettdigital · Jun 26, 2020 · 3bba512 · 3bba512
2 parents 71d7b6a + 54f9204
commit 3bba512
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/onepassword.py b/onepassword.py
@@ -134,7 +134,8 @@ def get_access_token(self, secret, shorthand):
                  f"{self.op} signin {secret['signin_address']} {secret['username']} {secret['secret_key']} "
                  f"--output=raw --shorthand={shorthand}"),
                 shell=True,
-                capture_output=True
+                capture_output=True,
+                env=os.environ
             )
             process.check_returncode()
             return process.stdout.decode('UTF-8').strip()

diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "onepassword"
-version = "0.3.2"
+version = "0.4.0"
 description = "Python wrapper for the 1password CLI"
 
 repository = "https://github.com/lettdigital/onepassword-python"