feature: commands to check if logged in about cli

[kspearrin]

Resolved in #47

[Mange]

That would be great. I had to end up guess from the error message string in my script, which is very brittle.

[blacs30]

Would like to see it too, would be very helpful

[ewjoachim]

Integrating the cli from a Python script and wondering all the same.

That being said, there has to be some information persisted on the disk when bitwarden is logged in, maybe we could check for that (while waiting for a proper --check to be implemented)

[ewjoachim]

Bingo :) Took me a while to explore the code base but allow me the occasion to say once I had found the jslib repo, it was really nice and readable. Many pointers, but at least, each module seems to have a unique dedicated purpose. Thank you.

cli/src/bw.ts

Lines 67 to 75 in 783e7fc

	if (process.env.BITWARDENCLI_APPDATA_DIR) {
	p = path.resolve(process.env.BITWARDENCLI_APPDATA_DIR);
	} else if (process.platform === 'darwin') {
	p = path.join(process.env.HOME, 'Library/Application Support/Bitwarden CLI');
	} else if (process.platform === 'win32') {
	p = path.join(process.env.APPDATA, 'Bitwarden CLI');
	} else {
	p = path.join(process.env.HOME, '.config/Bitwarden CLI');
	}

You can check <dir>/data.json and search for a key "userId". If it's there, user is logged. If it's not (or if file doesn't exist), user is not.

[ewjoachim]

Hahaha the doc is actually giving this information too. I found it by googling for BITWARDENCLI_APPDATA_DIR 😂

https://help.bitwarden.com/article/where-is-data-stored-computer/#cli

(it's interesting to note that the doc is not synced with the code regarding windows location. Let's believe the code)

[andykais]

in that case, I would still like to request that either a login check command is added, or the ability to ask the cli where the appdata is. A program built on top of bw should not have to keep track of four possible locations of internal config data. If its allowed, I am happy to take a stab at this and PR something

[ewjoachim]

(yes, of course, as mentionned, the solution I described was just a workaround)

[ewjoachim]

Yay thank you and @estheruary 👏 🎉 👍 ! Any idea when that might land in a release ?