Hello all. New here, just subscribed to the list :)

I used the keepassx2pass.py script recently to import my keepassx data
to pass, and I noticed that it didn't handle forward slashes. As a
result, when a title like 'mail/messaging' was given to pass it created
a directory 'mail' and an entry 'messaging'. What it should do instead
would be to create an entry like 'mail-messaging'.

So I fixed that. I made it replace forward slashes with dashes. I have
attached a patch.
>From 39e2af5fb2034c387d3421f0605c1b586a05df88 Mon Sep 17 00:00:00 2001
From: George Angelopoulos <[email protected]>
Date: Wed, 23 Apr 2014 17:04:54 +0300
Subject: [PATCH] keepassx2pass: handle forward slash '/' in titles

Without this, a forward slash in the title creates a new directory
in the password-store. This replaces forward slashes with dashes.
---
 contrib/importers/keepassx2pass.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/importers/keepassx2pass.py b/contrib/importers/keepassx2pass.py
index dc4b1e5..f9384d2 100755
--- a/contrib/importers/keepassx2pass.py
+++ b/contrib/importers/keepassx2pass.py
@@ -26,7 +26,7 @@ def space_to_camelcase(value):
 
 def cleanTitle(title):
     # make the title more command line friendly
-    title = re.sub("(\\|\||\(|\))", "-", title)
+    title = re.sub("(\\|\||\(|\)|/)", "-", title)
     title = re.sub("-$", "", title)
     title = re.sub("\@", "At", title)
     title = re.sub("'", "", title)
-- 
1.9.2

_______________________________________________
Password-Store mailing list
[email protected]
http://lists.zx2c4.com/mailman/listinfo/password-store

Reply via email to