WebhookService has no timeout on outgoing HTTP requests

[meshackyaro]

Description

WebhookService.send() (src/webhook/webhook.service.ts) builds a raw http.request()/https.request() call with no timeout option set and no req.setTimeout(...) call. If a registered webhook endpoint accepts the TCP connection but never responds (or responds arbitrarily slowly), the request hangs indefinitely — sendWithRetry()'s per-attempt delay (1000 * (i + 1)) only kicks in after a request settles (resolves or rejects), so a single unresponsive endpoint can stall dispatch to that endpoint far longer than intended, and since dispatch() awaits Promise.allSettled across all endpoints, a slow endpoint delays the overall dispatch() call's resolution for every caller that awaits it (e.g. EscrowController.raiseDispute, every worker's sweep loop).

Component

Backend

Difficulty

🟢 Easy

Tasks

• Add an explicit timeout to the http/https request in WebhookService.send() (e.g. req.setTimeout(ms, () => req.destroy(new Error('timeout'))))
• Choose and document a sensible default timeout, ideally configurable via an env var
• Add a test using a mock server that never responds, asserting the request fails after the configured timeout rather than hanging

Acceptance Criteria

• An unresponsive webhook endpoint fails fast after a bounded timeout instead of hanging indefinitely
• A test proves the timeout behavior

Estimated Time

2-4 hours

[Nathanpvd]

@Nathanpvd has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

I’ve looked through the issue requirements and feel prepared to implement a solution that aligns with the proposed design and acceptance criteria.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Nathanpvd to this issue.

[leadgenexpert454]

@leadgenexpert454 has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi maintainer
I’d love to help out with this issue. I’ve checked the current codebase/documentation and know exactly what needs to be updated to make this clearer. I will ensure everything aligns perfectly with your existing contribution guidelines.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @leadgenexpert454 to this issue.

[Suzan-max]

@Suzan-max has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi, I'd like to work on this issue. I've gone through the description and acceptance criteria and I'm confident I can deliver it. I'll make sure every acceptance criterion is met and submit a clean, tested PR. Please assign it to me.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Suzan-max to this issue.

[paulddeveloper-droid]

@paulddeveloper-droid has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

I’d like to work on this issue. I’ve reviewed the requirements and understand the expected changes. I’ll follow the existing project patterns, add the required tests, and keep the implementation within the stated scope. I’m ready to start.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @paulddeveloper-droid to this issue.

[TS-mfon]

@TS-mfon has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Gm
Please can i work on this issues

it would be an opportunity to contribute to your project

ℹ️ Repo Maintainers: To accept this application, review their application or assign @TS-mfon to this issue.

[drips-wave]

Congratulations, @Suzan-max! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on August 31, 2026.

🧑‍💻 @Suzan-max: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

Warning

When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

🌊 Happy Wave 🌊