feat(webhook): make sure webhook urls are valid

[laurentlp]

This PR adds a check to make sure that the synced webhook URLs are valid URIs and are reachable using an option request.

Closes MES-125

[linear]

MES-125 Add validation for webhook urls

We should send 400 errors if urls for webhooks aren't valid https urls.

[samuelmasse]

It looks like there is a problem with Jest and importing other packages from yarn workspaces.

src/channels/base/channel.ts:1:22 - error TS2307: Cannot find module '@botpress/messaging-base' or its corresponding type declarations.

    1 import { uuid } from '@botpress/messaging-base'
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~

[samuelmasse]

Building the repo first seems to solve the problem. I think rewiring might also solve it. You can see how the rewiring works when working with ts-node in the rewire.ts. Ideally we would need to require that file before every test.

[samuelmasse]

We need to send a 400 with a message back to the user otherwise great work!

[laurentlp]

Building the repo first seems to solve the problem. I think rewiring might also solve it. You can see how the rewiring works when working with ts-node in the rewire.ts. Ideally we would need to require that file before every test.

kulshekhar/ts-jest#1648

[samuelmasse]

👍