Fields are not comma escaped

[Evidlo]

My passwords are randomly generated by pass and so some of them contains commas. I'm able to correct the few that are wrong by hand, but it would be nice to have these commas escaped.

On an unrelated note, this script is useful for converting to keepass as well, since it supports csv import.

[d4ndo]

I would rather not modify the password. This could mess up a lot of things.
According to http://keepass.info/help/kb/imp_csv.html commas are not even escaped.

I could add a delimiter/separator variable so you could change the »comma« separator to a »whitespace« separator. No password should contain a whitespace or tab. This makes it easy to import to libreoffice and do modifications.

P.S. Password gorilla complains if to many commas are set.

[Evidlo]

Yes that would be useful.

[ruudgrosmann]

I came across this bug too. I solved it by editing the script. It solves some other bugs too (double quotes in passwords or other fields give the same problem as comma's do; entry's with the same name, but another user name overwrite each other). The structure of the groups in the pwgorilla database is reflected in the qtpass structure.

#!/usr/bin/env ruby

# Copyright (C) 2013 David Sklar <david.sklar@gmail.com>. All Rights Reserved.
# This file is licensed under the GPLv2+. Please see COPYING for more information.

def uniek (keys, key)
	while (keys.key? key)
		if (key.gsub!( /(\d+)$/) do |hit|
				($1.to_i + 1).to_s
			end)
		else
			key += '1'
		end
	end
	key
end

entries = {}

def escapeline (str)
	str = str.gsub( /(^|,)"/, '\1°ESC°').gsub( /"(,|$)/, '°ESC°\1')
	str.gsub( /°ESC°([^°]*)°ESC°/) do |hit|
		$1.gsub(/,/, '&comma;').gsub( /""/, '&dquot;')
	end
end
def unescapestring (str)
	str.gsub( /&dquot;/, '"').gsub( /&comma;/, ',')
end

$stdin.each do |line|
	uuid, group, title, url, user, password, notes =
			escapeline( line).strip.split(',').map { |e| unescapestring e }
	next if uuid == "uuid"

	prefix = (group.gsub( /\./, '/') + '/' + title.gsub( /['\"()!]/,''))
	prefix.gsub!( /\s/, '_')
	prefix = uniek( entries, prefix)
	entries["#{prefix}"] = password + "\n";

	if user && user.length > 0
		entries["#{prefix}"] += "user: #{user}\n";
	end
	if url && url.length > 0
		entries["#{prefix}"] += "url: #{url}\n";
	end
	if notes && notes.length > 0
		entries["#{prefix}"] += "notes: #{notes.gsub('\n',"\n").strip}\n"
	end
end

pass_top_level = "gorilla"
entries.keys.sort.each do |k|
	print "#{pass_top_level}/#{k}......"
	IO.popen("pass insert -m -f '#{pass_top_level}/#{k}' > /dev/null", 'w') do |io|
		io.puts entries[k] 
	end

	if $? == 0
		puts " done!"
	else
		puts " error!"
	end
end

In the configuration, in the template tab, add url and user, because these fields are taken from the pwgor-csvfile and added to the entry.