-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Android push_token field is empty #4296
Comments
Ah, I think the problem is that we try to register on login before the device token is saved, so this warning will be shown even on successful registrations. Can you check the database if the fields really are empty?
|
It's not empty:
|
@stefan0xC, any ideas on how to fix this? |
Not really. We could remove the warnings? |
Only if that warning is false. Else it is good i think. I really need to start working on my admin updates again, that would also help. |
Removing the warning would not solve the problem that push notifications will work or? |
This comment was marked as outdated.
This comment was marked as outdated.
It's not false per se but misleading because I was not taking the order of operations in account but only thinking about users that connected a device previously to #3792 and not the common case. Currently, if push notifications are setup a new device will
|
Oh, looking at the code, I think we can simply fix this by not calling the vaultwarden/src/api/identity.rs Line 251 in ad1d65b
|
don't try to register a push device when the device is new it will be registered when the push token is saved fixes dani-garcia#4296
Now we just need to test the fix. |
Push notifications should work (they do on my test system). If they still don't on yours even though the You can test if the push notifications work by adding/renaming a folder in the web-vault and see if the change is auto-synchronized to the android app after a few seconds. |
Hello, I seem to be in the same situation, that is, I see the warning when I make the first login with my Android phone running Bitwarden from the Play Store.
Oh, am I supposed to see some log lines to indicate this push activity as well? (Other than that warning) I did try to add a folder and a note from the web interface, however they didn't automatically replicate to the Bitwarden app that was open in front of me on my phone. Note: I run Archlinux's vaultwarden package (1.30.2-1) and the push stuff is configured to use the *bitwarden.eu servers. |
Not with |
don't try to register a push device when the device is new it will be registered when the push token is saved fixes #4296
I started to get this error since I changed the notification server from .com to .eu with the last update |
If you switched, you also need to get new keys. And i think you should deregister your mobile devices, fully logout, clear data, and login again. Else they are probably not registered at the eu servers. |
Okay I had tested everthing and it's working fine, |
Subject of the issue
I followed up the wiki to enable push notifications.
Over web I can confirm that websocket is still working:
but im running into the issue push_token is empty
Deployment environment
Your environment (Generated via diagnostics page)
Config (Generated via diagnostics page)
Show Running Config
Environment settings which are overridden: DOMAIN, SIGNUPS_ALLOWED, INVITATIONS_ALLOWED, ADMIN_TOKEN, SMTP_HOST, SMTP_SECURITY, SMTP_PORT, SMTP_FROM, SMTP_FROM_NAME, SMTP_USERNAME, SMTP_PASSWORD
Install method: Docker image
Clients used: Android
Reverse proxy and version: nginx latest debian
Steps to reproduce
Enable push notification like described at the wiki
Try to reconnect an android client
Expected behaviour
push_token should be provided
Actual behaviour
no push token is given
The text was updated successfully, but these errors were encountered: