gopassbridge icon indicating copy to clipboard operation
gopassbridge copied to clipboard

gopassbridge prompting me to update gopass despite having newer version

Open skewballfox opened this issue 2 years ago • 6 comments

something seems to be messing up with this function

when I open gopassbridge, instead of showing my suggested passwords, I'm prompted to update gopass:

Please update gopass to version 1.8.5 or newer.

however I'm currently on gopass 1.14.1.

let me know if you need anything else

skewballfox avatar May 24 '22 20:05 skewballfox

I think I might have a simple solution if you're also experiencing this issue with the version that's currently available in the Fedora repos. I uninstalled that version and downloaded the rpm from the Github releases page, installed it, and ran the configure option again. After that the extension in Firefox no longer tells me that gopass need to be newerd than 1.8.5

jwmurraycell avatar Jun 02 '22 21:06 jwmurraycell

Thanks for the suggestion - but I've tried that (now gopass 1.14.3 go1.18.2 linux amd64), re-ran gopass-jsonapi configure, and restarted firefox - but still get the same error.

I have then did the same thing for gopass-jsonapi itself, and now it works :+1: (weirdly, dnf said Downgrade to gopass-jsonapi-1.11.1-1.x86_64)

tennox avatar Jun 16 '22 09:06 tennox

so this is an issue with the version of gopass-jsonapi provided in the fedora repositories? should I submit a bug report asking them to upgrade that package specifically?

skewballfox avatar Jul 29 '22 17:07 skewballfox

A nice way to debug this would be to modify the gopass-bridge.sh accordingly:

--- gopass_wrapper.sh	2022-08-25 00:39:31.368000000 +0200
+++ gopass_wrapper.sh	2022-08-25 00:40:09.696000000 +0200
@@ -10,6 +10,6 @@ fi
 export PATH="$PATH:/usr/local/bin" # required on MacOS/brew
 export GPG_TTY="$(tty)"
 
-gopass-jsonapi listen
+tee -a /tmp/gopass-in | gopass-jsonapi listen | tee -a /tmp/gopass-out
 
 exit $?

Use hexdump -C on /tmp/gopass-in vs /tmp/gopass-out.

Most likely getVersion is messed up.

isodude avatar Aug 24 '22 22:08 isodude

Oh, and it's probably related to https://github.com/gopasspw/gopass-jsonapi/issues/2 ?

isodude avatar Aug 24 '22 22:08 isodude

@isodude made me think of a workaround. You can just patch the result of getVersion in the wrapper script:

-gopass-jsonapi listen
+gopass-jsonapi listen | sed 's_version":"0.0.0","major":0,"minor":0,"patch":0_version":"1.8.5","major":1,"minor":8,"patch":5_'

But looking at this I would suspect a packaging issue.

lkiesow avatar Jul 05 '23 22:07 lkiesow