Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cond: Wrong type argument: stringp, nil #20

Open
jsilve24 opened this issue Dec 19, 2023 · 1 comment
Open

cond: Wrong type argument: stringp, nil #20

jsilve24 opened this issue Dec 19, 2023 · 1 comment

Comments

@jsilve24
Copy link
Contributor

I started getting this warning after upgrading to newest version (on AUR) of bw-cli and emacs-bitwarden. I tracked the error down.

Relevant error in bw-cli
bitwarden/clients#7126

What this is causing:

The bw cli is giving the following error:

Error: Lock file is already being held
    at /usr/lib/node_modules/@bitwarden/cli/node_modules/proper-lockfile/lib/lockfile.js:68:47
    at callback (/usr/lib/node_modules/@bitwarden/cli/node_modules/graceful-fs/polyfills.js:306:20)
    at FSReqCallback.oncomplete (node:fs:211:5) {
  code: 'ELOCKED',
  file: '/home/jds6696/.config/Bitwarden CLI/data.json'
}

Node.js v18.18.2

which is causing exit-code to equal 1 and within the bitwarden-runcmd function which then gets pickuped but misshandled within the follwing if form

(if (eq exit-code 0)
output
(cond ...)) 

What works for me is replacing (eq exit-code 0) with t for the moment.

I am sure there is a better solution that bypasses this error and also doesn't simply remove the functionality of this clause but this worked for me.

Hope this helps.

jsilve24 added a commit to jsilve24/emacs-bitwarden that referenced this issue Dec 19, 2023
@seanfarley
Copy link
Owner

Thanks for the report! I just noticed this as well. I've been wanting to rewrite this package to use the REST server that is built into bw-cli which would circumvent a lot of these types of errors. Just need more hours in the day 😞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants