Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Failed sync reported as successful #12

Closed
paescuj opened this issue May 31, 2018 · 5 comments
Closed

Failed sync reported as successful #12

paescuj opened this issue May 31, 2018 · 5 comments
Labels

Comments

@paescuj
Copy link

paescuj commented May 31, 2018

Since I'm using bitwarden in conjunction with a self signed certificate the sync doesn't work (without the workaround mentioned in #9).
However it is still reported as successful:

$ bw sync --session ***
Syncing complete.

Interestingly, the last sync date is correct / doesn't get updated:

$ bw sync --last --session ***
2018-05-28T15:49:04.015Z
@edermi
Copy link

edermi commented Nov 20, 2018

I stumbled across the same issue and can confirm this issue. I would expect the CLI to print an error message like the log-in function already does.

@kspearrin
Copy link
Member

I'm curious how you were able to log in and obtain a session without a cert issue in the first place?

@edermi
Copy link

edermi commented Nov 20, 2018

NODE_EXTRA_CA_CERTS=<path to my ca> bw login from #9 works, but without the environment variable there is an error stating that the cert is self-signed. bw sync won't report this error and falsely state that the sync was successful. Using NODE_EXTRA_CA_CERTS=<path to my ca> bw sync works correctly

@ghost
Copy link

ghost commented Dec 2, 2019

We can confirm the same issue. Another hack to make it work is to set the environment variable:
NODE_TLS_REJECT_UNAUTHORIZED=0

Since this product is specifically in the security domain, we would not expect workarounds regarding chains of trust to get it to actually work.

The first fix should be to get an error code and message when a sync fails, that should be easy enough?

@bitwarden-bot
Copy link

Hi paescuj,
We're cleaning up our repositories in preparation for a major reorganization. Issues from last year will be marked as stale and closed after two weeks. If you still need help, comment to let us know and we'll look into it.
Thanks!

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

No branches or pull requests

5 participants