Replies: 1 comment 5 replies
-
Arguing for/against push notifications is largely equivalent to arguing for/against WebSockets (i.e., wanting one but not the other is almost logically inconsistent since they solve the same problem for the different kind of clients). The only reason I use the adverb "largely" is that push notifications require registering with the Bitwarden API which may cause more overhead and may be a privacy concern. Technically, it's possible for a client to update information overwriting information written from another client if the data was out-of-sync; but that's unlikely since the changes would have to be 1 second apart due to this diff several years ago. Since nothing is instantaneous anyway, I'm sure there is a time interval where such overwrites can still occur even with WebSockets or push notifications. I suppose if the system time on the server or client is off, then this increases the chances as well. Personally, I don't use either since I prefer the reduction in code and am content with this 1-second window (which can be reduced on a local clone of this repo) especially since clients and server all use the same time source—forced by clients having to use an internal network to access the Vaultwarden server, and the router redirecting all NTP requests from all devices (including Bitwarden clients and Vaultwarden server) on my internal L2/L3 networks to a self-hosted NTP server. |
Beta Was this translation helpful? Give feedback.
-
I am not sure whether I should enable push notifications. Is the overhead worth it?
I've been happy without the push notifications until now. When I added or updated data in web/desktop/browser, it was replicated almost instantly. That's all I needed, since it's not that hard to use the pull down gesture to sync on mobile.
I also know that they are a requirement for logging in with a device.
Have you enabled push notifications? What are your opinions?
Beta Was this translation helpful? Give feedback.
All reactions