slovensko-digital / govbox-pro

Nová, krajšia a lepšia schránka.
https://sluzby.slovensko.digital/govbox-pro/
European Union Public License 1.2
2 stars 2 forks source link

Add webhooks in automation rules #455

Closed celuchmarek closed 3 months ago

celuchmarek commented 3 months ago

Testy dorobím, naprjv ale chcem vedieť, či je to dobre takto. V podstate som do Automation rules pridal ďalšie triggery z EventBus (message_thread_changed, message_thread_created, message_updated).

Tie webhooky vedia byť prázdny POST na zadanú URL alebo obsahujú body podľa standard-webhooks. T.j. napr:

{
  "type":"MessageThread.message_thread_changed",
  "timestamp":"2024-07-19T10:32:49.129+02:00",
  "data": {
    "message_thread_id": 251
  }
}

{
  "type":"Message.message_created",
  "timestamp":"2024-07-19T10:44:04.224+02:00",
  "data": {
    "message_id": 725,
    "message_thread_id": 251
  }
}

Autorizáciu webhookov som zatiaľ tiež neriešil, ale IMHO to zatiaľ netreba. Ale je to %lahké dorobiť - do nejakého headera pridáme podpis, prípadne jwt.

jsuchal commented 3 months ago

@celuchmarek tu este dve veci mozno, co by sme rovno mohli skusit ako test toho ci to funguje dobre:

  1. overit scenar so zmenou statusu vlakna (co chce klient)
  2. ked vznikne nove vlakno u nas v ssd, tak nech to posles push a v slacku sa to nejako zobrazi - napada mi - ked je nieco na podpis, nech to prileti do slacku.
celuchmarek commented 3 months ago

Snímka obrazovky 2024-07-23 o 11 21 28

celuchmarek commented 3 months ago

@jsuchal updatol som podľa review a podľa dohody. Idem ešte nejaký test spraviť.

jsuchal commented 3 months ago

@celuchmarek swagger nejaky a/alebo pouzivatelsku prirucku netreba upravit?

celuchmarek commented 3 months ago

@jsuchal pridal som do swaggeru. Chceme aj do tej PDF dokumentácie?