-
Notifications
You must be signed in to change notification settings - Fork 63
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
hub.register(token, tag); throws exception if already registered #22
Comments
Hi, can you provide any more detail? Maybe a callstack or simple code sample? |
//issue: fcm push messaging is working properly. hub.register works the first time you launch the app. Every subsequent time it crashes. In a previous version we would call hub.reg with a new tag and it would 'update' the registration automatically (that's how we do it on iOS too) CODE: public class MyInstanceIDService extends FirebaseMessagingService {
} STACK |
I'm seeing an error with a similar call to hub.register(token) as well. Not sure if it's related to this issue. I'm seeing "com.microsoft.windowsazure.messaging.NotificationHubUnauthorizedException: Unauthorized" returned from "return registerInternal(registration);" of notification-hubs-android-sdk-0.6/NotificationHub.java This was working previously. |
@hangryfabian We were not able to reproduce this issue on our end. Are you still experiencing this issue on the new version of the SDK? |
hub.register() crashes the second time (i.e. you cannot run this or change the registered tag every time on launch).
This used to work in a previous version, where we can re-register a new tag when a user logged out of their account and registered/signed in with a new one.
We are handling this by storing the FCM tokens, but this function should be failing gracefully, or calling a update registration with new tag instead. It shouldn't throw a random exception.
Thanks!
The text was updated successfully, but these errors were encountered: