Obtener el usuario objetivo para eventos de webhook de notificación

Desarrollador
1

Veo que los webhooks ahora admiten notificaciones, lo cual es fantástico. Es algo que realmente necesitamos. Sin embargo, el payload carece de una pieza crítica de información: el usuario para el que está destinada la notificación. En este ejemplo de payload podemos ver al autor del post que activó la notificación, pero no al usuario que debería recibirla.

{
  "notification": {
    "id": 119,
    "notification_type": 9,
    "read": false,
    "created_at": "2019-09-18T13:42:21.248Z",
    "post_number": 1,
    "topic_id": 75,
    "fancy_title": "Phallguy should get a notice about this",
    "slug": "phallguy-should-get-a-notice-about-this",
    "data": {
      "topic_title": "Phallguy should get a notice about this",
      "original_post_id": 111,
      "original_post_type": 1,
      "original_username": "Paul_Alexander",
      "revision_number": null,
      "display_username": "Paul_Alexander"
    }
  }
}

He revisado si existe una API para obtener el usuario destinatario a partir del ID de la notificación, pero no he encontrado nada documentado. ¿Existe alguna forma de obtener el nombre de usuario o ID del destinatario en el payload de la notificación?

Para dar contexto, estamos intentando consolidar las notificaciones de varios sistemas en una experiencia única para nuestros usuarios.

2

@Falco or @blake do you have any thoughts on how I can figure out which user a notification is for when received through the new notification webhook?

3

Just looked at this briefly to see if there was a quick answer, and you are right, it does not look like we have a get api endpoint for fetching an individual notification by id, but that wouldn’t be very efficient to call every time anyways.

Most likely this will require a code change to the notification serializer to include the user_id/username inside of the notification payload, so that a separate api request wouldn’t be necessary.

4

We’re using the hosted service at the moment otherwise I’d build my own plugin. Would you folks consider adding the user id/username to then notification payload in the webhook?

5

Did you check the headers?

6

I did - hoping to find some extra meta data in there but didn’t find anything user specific.

Request URL: https://XXX
Request method: POST
Accept: */*
Connection: close
Content-Length: 420
Content-Type: application/json
Host: XXX
User-Agent: Discourse/2.4.0.beta4
X-Discourse-Instance: https://XXX
X-Discourse-Event-Id: 11
X-Discourse-Event-Type: notification
X-Discourse-Event: notification_created
X-Discourse-Event-Signature: sha256=XXX
7

Checking in to see if you folks have any thoughts/updates. The notification webhook without a target user isn’t really of much use.

8

Hey guys have you had a chance to review the issue? Is there any hope that we can get the target user id in the notification event?

9

I’ve created a pr that will add the user_id to the notification webhook json payload:

10

Hola @blake, ¿sería posible añadir también el user_external_id? Trabajamos con el SSO de Discourse y no tenemos realmente los user_ids de Discourse en nuestro sistema.

11

¡Hola Albert!

Creo que es una buena idea y he creado un commit que añadirá el external_id del usuario si SSO está habilitado.

¡Espero que esto ayude! Se añadirá a tu sitio la próxima vez que se implemente, lo cual debería ser en algún momento de esta semana.

12

¡Vaya, qué rápido! Muchas gracias, se agradece mucho :smiley: ¿Sabes cuál es el tiempo estimado aproximado para que esos cambios lleguen a los servidores de Discourse alojados?

13

Oh, olvídalo, no leí la última frase :sweat_smile:. ¡Gracias de nuevo!!