serafeimo Comments - Answer Overflow

Topics

serafeimo

•Created by serafeimo on 9/27/2023 in #💬│support

ReplyError: CROSSSLOT Keys in request don't hash to the same slot

Hi all! Any feedback on the above?

9 replies

•Created by serafeimo on 9/27/2023 in #💬│support

ReplyError: CROSSSLOT Keys in request don't hash to the same slot

From a quick investigation we had it seems that when Novu runs with Redis in cluster mode, it fails because it searches for the Redis host property (which is the single host). If we set this property in order to proceed, then it fails to send the keys due to the above reason (we suspect because it uses the single host with priority for some reason). I don't know if what I described makes sense but we haven't managed to find a solution apart from changing back to a single host. Do you know if the cluster mode is tested and working?

9 replies

•Created by serafeimo on 9/27/2023 in #💬│support

ReplyError: CROSSSLOT Keys in request don't hash to the same slot

According to this page https://hackernoon.com/resolving-the-crossslot-keys-error-with-redis-cluster-mode-enabled the problem could be solved by adding a prefix in the keys. Does the REDIS_CLUSTER_KEY_PREFIX property have to do with the above issue?

9 replies

•Created by serafeimo on 5/3/2023 in #💬│support

Activity feed empty (v0.14.0)

Great, thanks for your support!

13 replies

•Created by serafeimo on 5/3/2023 in #💬│support

Activity feed empty (v0.14.0)

So, api service receives the notifications and the workers pick them up in order to process them, right? Are they retrieving them from redis or DB?

13 replies

•Created by serafeimo on 5/3/2023 in #💬│support

Activity feed empty (v0.14.0)

Thanks for your response @pablo.fernandez.otero. That seems a great enhancement. I've got a couple of questions about it. - Do we need to scale both api and worker services to increase performance or just one of them? - I guess that we continue to use api service's APIs in order to integrate with Novu as before (create subrscribers, trigger notifications, check activity, etc), right?

13 replies

•Created by serafeimo on 5/3/2023 in #💬│support

Activity feed empty (v0.14.0)

Well, I totally missed that. I'm sorry for that 😕 Is there any document that describes what the worker does and what is the connection among api, worker and web? I guess that we'll continue using the api service as an integration point with Novu, right?

13 replies

•Created by serafeimo on 5/3/2023 in #💬│support

Activity feed empty (v0.14.0)

Hi @Pawan Jain, I'm triggering the notification via the API call (/trigger) and I'm passing the api key which I got from Novu's web dashboard. Then I checked in the dashboard and by calling the /notifications API, again with the same api key.

13 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

So, it seems that when I have 2 tokens set in the subscriber, Novu uses FCM for both, regardless of the APNS's provider state (active/inactive)

56 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

Yes, FCM valid token and APNS valid token

56 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

Good morning! Another thing I noticed is the following: - Activate FCM - Activate APNS - Create a subscriber with both FCM and APNS tokens - When sending push notifications, Novu tries to send both notifications with FCM. So, for the second token it will get an error because it's not a valid FCM token but an APNS one. It looks like it gets the first active integration and use it for everything.

56 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

I've tested it further and it seems that it tries to send notifications using every active integration, regardless of the subscriber's credentials. So, for example: - FCM -> active - APNS -> active - Subscriber has only FCM token set in credentials. - Send push notification - 2 actions have been triggered: - sent with FCM successfully - failed to send with APNS (due to lack of token) -- From my point of view, the expected scenario would be to send one notification using FCM since this is what the subscriber has credentials for. Of course the above example is the same for the other way around (only apns token). I believe that the provider should be used according to the subscriber's credentials so that the system can send notifications to users that have only android or ios devices without the need of constantly activating/deactivating the providers. I think it's the normal and most common case and not an exceptional scenario.

56 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

I can't see an option in trigger endpoint. I can see only an API to update an integration (i.e. set it on/off) but in any case this will be an overkill to do for every notification. In addition, I'm sure that it will create problems when many notifications will being sent in parallel.

56 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

I don't think that it makes sense though. There are users that have android devices and others with ios devices and we need to send notifications to all of them. I guess that this is a pretty common case in general.

56 replies

•Created by serafeimo on 3/22/2023 in #💬│support

Issue while having both FCM and APNS active

No, I only have apns token set.

56 replies

•Created by serafeimo on 3/6/2023 in #💬│support

APNS error while sending notification (v 0.12.0)

Hi all! Is there any update on the release date? Thanks!

23 replies

•Created by serafeimo on 3/6/2023 in #💬│support

APNS error while sending notification (v 0.12.0)

That's great news. Thanks @biswaviraj ! Is there an expected date for this patch?

23 replies

•Created by serafeimo on 3/6/2023 in #💬│support

APNS error while sending notification (v 0.12.0)

@Pawan Jain I created 2 bugs for the above issues: https://github.com/novuhq/novu/issues/2966 https://github.com/novuhq/novu/issues/2967

23 replies

•Created by serafeimo on 3/6/2023 in #💬│support

APNS error while sending notification (v 0.12.0)

UPDATE: We discovered something interesting today. When we use a valid token, the notification is being received BUT we get an error (with an empty message) and the transaction is marked as failed.

{
  “stack”: “Error: \n    at APNSPushProvider.<anonymous> (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:84:35)\n    at step (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:44:23)\n    at Object.next (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:25:53)\n    at fulfilled (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:16:58)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)“,
  “message”: “”
}

When we use a wrong token then we get an error with the correct message and the transaction is also being marked as failed.

{
  “stack”: “Error: 85a57fd53908d108da4c3fe1da073677c418e3bb7674f96e2cb46bcc69eb8a3 failed for reason: BadDeviceToken\n    at APNSPushProvider.<anonymous> (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:84:35)\n    at step (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:44:23)\n    at Object.next (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:25:53)\n    at fulfilled (/usr/src/app/providers/apns/build/main/lib/apns.provider.js:16:58)\n    at runMicrotasks (<anonymous>)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)“,
  “message”: “85a57fd53908d108da4c3fe1da073677c418e3bb7674f96e2cb46bcc69eb8a3 failed for reason: BadDeviceToken”
}

So, in both cases the transaction is marked as failed but in the first case we actually saw it coming in.

23 replies