Fix 33 - migrate to setuptools

Author: hsanson

.gitignore

@@ -2,3 +2,4 @@ application/MANIFEST
 application/dist/*
 application/build/*
 extension.zip
+chrome_pass.egg-info

README.md

@@ -12,7 +12,7 @@ There are two folders in this repository that contain:
      and password store.
 
 To use the extension you need to install the extension in your chrome or
-chromium browser and the python native application (nativePass).
+chromium browser and the python native application (chrome_pass).
 
 ## Requirements
 
@@ -32,8 +32,8 @@ These instructions have been tested in Ubuntu 22.04 and later:
 ### Python native pass application install
 
     sudo apt-get install pass python3 python3-pip
-    pip3 install --user chrome-pass==0.4.0
-    nativePass install
+    pip install --user chrome-pass==0.5.0
+    chrome_pass install
 
 ### Chrome extension install
 
@@ -129,12 +129,14 @@ then load the path to the *extension* folder using the *Load unpacked extension*
 button. After the extension is loaded into Chrome take note of the *extension
 ID*.
 
-Next we need to install the *nativePass* wrapper script and install the Native
+Next we need to install the *chrome_pass* wrapper script and install the Native
 Host Application manifest:
 
     cd application
-    python3 setup.py install
-    nativePass install [extension ID]
+    pip install --upgrade setuptools build --user
+    python -m build
+    pip install . --user
+    chrome_pass install [extension ID]
 
 ## Usage
 
@@ -144,6 +146,26 @@ Host Application manifest:
   - You may type a search term in the search box to filter the list of usernames.
 - The form should be automatically filled with the username and corresponding password.
 
+## Version 0.5.0 Notes
+
+The `nativePass` script has been renamed to `chrome_pass`.
+
+Version 0.5.0 of chrome-pass uses setuptools instead of distutils to package and
+install the native application. When installing you may get errors such as:
+
+```
+ERROR: Cannot uninstall 'chrome-pass'. It is a distutils installed project and
+thus we cannot accurately determine which files belong to it which would lead
+to only a partial uninstall.
+```
+
+In this situation is necessary to manually uninstall older versions of the package:
+
+1. Remove `nativePass` script. Find it using `which nativePass`.
+2. Find where site-packages are installed (e.g.
+   /var/lib/python3.10/site-packages) and remove all `chrome_pass-0.X.0...`
+   files and directories.
+
 ## Troubleshooting
 
 If for some reason the extension is unable to get the list of usernames from
@@ -157,21 +179,21 @@ your password store the most probable reasons are:
  passwords and username list. This file is usually located at
  ~/.config/google-chrome/NativeMessagingHosts folder and MUST be named
  *com.piaotech.chrome.extension.pass.json*.
-- The nativePass script has a helper method to generate the native host
- manifest *nativePass install [extension id]* so use it to generate the
+- The chrome_pass script has a helper method to generate the native host
+ manifest *chrome_pass install [extension id]* so use it to generate the
  manifest. If you do not give it am [extension id] it will generate the
  manifest with the id of the extension from the chrome web store.
 - Another possible issue is that the manifest contents does not match your
   system:
   - Ensure the *path* contains the absolute path to the location of the
-    nativePass wrapper script.
+    chrome_pass wrapper script.
   - Ensure the *allowed_origins* contains the URI with the exact extension ID
     installed in Chrome. To get the extension ID simply browse chrome:
     //extensions and look for the ID of the chrome-pass extension installed.
 
 ## Note about python-gnupg
 
-It has been found that the nativePass application is unable to decrypt the gpg
+It has been found that the chrome_pass application is unable to decrypt the gpg
 passwords with some newer versions of python-gnupg. I can verify that the plugin
 works without issues when using gnupg module version 0.3.9 found by default in
 Ubuntu 16.04LTS.

application/nativePass application/chrome_pass/__init__.pyapplication/nativePass renamed to application/chrome_pass/__init__.py

@@ -1,6 +1,7 @@
-#!/usr/bin/env python3
+"""
+chrome_pass native application module
+"""
 
-# Requires python-gpg library
 import os
 import re
 import sys
@@ -9,18 +10,20 @@
 import shutil
 import difflib
 import pathlib
+import posixpath
 from urllib.parse import parse_qs
 from urllib.parse import urlparse
 from collections import OrderedDict
+from importlib.metadata import entry_points
 import pyotp
 import gnupg
 
 if sys.platform == "win32":
     # Interacts with windows registry to register this app
-    import winreg
+    import winreg  # pylint: disable=import-error
     # On Windows, the default I/O mode is O_TEXT. Set this to O_BINARY
     # to avoid unwanted modifications of the input/output streams.
-    import msvcrt
+    import msvcrt  # pylint: disable=import-error
     msvcrt.setmode(sys.stdin.fileno(), os.O_BINARY)
     msvcrt.setmode(sys.stdout.fileno(), os.O_BINARY)
 
@@ -97,13 +100,16 @@ def read_data(path):
     raise RuntimeError(f'Failed to decrypt {txt}')
 
 
-# Returns a dictionary with id and value pairs read from pass files that match
-# pattern:
-#
-#   key=value
-#
-# key only matches alphanumeric characters and cannot have spaces.
 def get_creds(path):
+    """
+    Returns a dictionary with id and value pairs read from pass files
+    that match pattern:
+
+      key=value
+
+    key only matches alphanumeric characters and cannot have spaces.
+    """
+
     # Read decripted pass file data.
     data = read_data(path).decode('utf-8').split("\n")
 
@@ -131,17 +137,22 @@ def get_creds(path):
     return creds
 
 
-# Sends the response message with the format that chrome HostNativeApplications
-# expect.
 def send_message(message):
+    """
+    Sends response messages in format compatible with chrome
+    HostNativeApplications.
+    """
     response = json.dumps(message).encode('utf-8')
     sys.stdout.buffer.write(struct.pack('I', len(response)))
     sys.stdout.buffer.write(response)
     sys.stdout.buffer.flush()
 
 
-# Method that implements Chrome Native App protocol for messaging.
 def process_native():
+    """
+    Method that implements Chrome Native App protocol to enable
+    communication between chrome-pass chrome extension and pass.
+    """
     size = sys.stdin.buffer.read(4)
 
     if not size:
@@ -173,21 +184,28 @@ def process_native():
         send_message({"action": "error", "msg": sys.exc_info()[0]})
 
 
-# Method prints to stdout the list of passwords ordered by a similarty pattern
 def print_list(pattern):
+    """
+    Method prints to stdout the list of passwords ordered by a similarty
+    pattern
+    """
     for credential in get_list(pattern)[:20]:
         print(credential)
 
 
-# Method prints to stdout the first match creds data.
 def print_creds(pattern):
+    """
+    Method prints to stdout the first match creds data.
+    """
     for credential in get_list(pattern)[:20]:
         account = credential[0] + "/" + credential[2]
         print(f'{get_creds(account)}')
 
 
-# Determines the path were the native app manifest should be installed.
 def native_path_chrome():
+    """
+    Determines the path were the native app manifest should be installed.
+    """
     if sys.platform == "darwin":
         return os.path.expanduser(
             '~'
@@ -228,8 +246,23 @@ def native_path_brave():
     sys.exit(1)
 
 
-# Installs the Native Host Application manifest for this script into Chrome.
+def find_chrome_pass_path():
+    """
+    Convoluted function to figure out the absolute path of the chrome_pass
+    console script.
+    """
+    entry_point = entry_points().select(
+            name='chrome_pass', group='console_scripts')[0]
+    package_path = list(filter(
+        lambda file: file.name == "chrome_pass",
+        entry_point.dist.files))[0]
+    return posixpath.abspath(package_path.locate())
+
+
 def install(native_path, extension_id):
+    """
+    Installs the Native Host Application manifest for this script into Chrome.
+    """
     if sys.platform == "win32":
         # Appends APPDATA to native_path and set this path as a registry value
         reg_key = os.path.join("Software", native_path)
@@ -242,14 +275,14 @@ def install(native_path, extension_id):
         os.makedirs(native_path)
 
     if sys.platform == "win32":
-        batch = "python \"{}\" %*".format(os.path.realpath(__file__))
+        batch = f"python \"{os.path.realpath(__file__)}\" %*"
         native_app = EXTENSION_NAME + '.bat'
         outfile = os.path.join(native_path, native_app)
         with open(outfile, 'w', encoding="utf-8") as file:
             file.write("@echo off\n\n")
             file.write(batch)
     else:
-        native_app = os.path.realpath(__file__)
+        native_app = find_chrome_pass_path()
 
     manifest = OrderedDict()
     manifest['name'] = EXTENSION_NAME
@@ -264,22 +297,23 @@ def install(native_path, extension_id):
         json.dump(manifest, file, indent='\t')
 
 
-if len(sys.argv) > 1:
-    if sys.argv[1].startswith('chrome-extension://'):
-        process_native()
-    elif sys.argv[1] == "install":
-        if len(sys.argv) > 2:
-            install(native_path_chrome(), sys.argv[2])
-            install(native_path_chromium(), sys.argv[2])
-            install(native_path_brave(), sys.argv[2])
+def run():
+    if len(sys.argv) > 1:
+        if sys.argv[1].startswith('chrome-extension://'):
+            process_native()
+        elif sys.argv[1] == "install":
+            if len(sys.argv) > 2:
+                install(native_path_chrome(), sys.argv[2])
+                install(native_path_chromium(), sys.argv[2])
+                install(native_path_brave(), sys.argv[2])
+            else:
+                install(native_path_chrome(), EXTENSION_ID)
+                install(native_path_chromium(), EXTENSION_ID)
+                install(native_path_brave(), EXTENSION_ID)
+        elif sys.argv[1] == "pass":
+            if len(sys.argv) > 2:
+                print_creds(sys.argv[2])
+        elif sys.argv[1] == "gpgbin":
+            print(f"GPG Binary path: {get_gpg_bin()}")
         else:
-            install(native_path_chrome(), EXTENSION_ID)
-            install(native_path_chromium(), EXTENSION_ID)
-            install(native_path_brave(), EXTENSION_ID)
-    elif sys.argv[1] == "pass":
-        if len(sys.argv) > 2:
-            print_creds(sys.argv[2])
-    elif sys.argv[1] == "gpgbin":
-        print(f"GPG Binary path: {get_gpg_bin()}")
-    else:
-        print_list(sys.argv[1])
+            print_list(sys.argv[1])

application/chrome_pass/__main__.py

@@ -0,0 +1,7 @@
+"""
+Console script main
+"""
+from . import run
+
+if __name__ == '__main__':
+    run()

application/chrome_pass/__pycache__/__init__.cpython-310.pyc

@@ -0,0 +1,19 @@
+[build-system]
+requires = ["setuptools"]
+build-backend = "setuptools.build_meta"
+[project]
+name = "chrome_pass"
+version = "0.5.0"
+authors = [
+  {name = "Horacio Sanson", email = "hsanson@gmail.com"},
+]
+description = "Native application for chrome-pass extension"
+requires-python = ">=3.8"
+keywords = ["pass", "chrome"]
+license = {text = "MIT"}
+dependencies = [
+  "python-gnupg",
+  "pyotp"
+]
+[project.scripts]
+chrome_pass = "chrome_pass:run"