refactor: Livechat webhook retry & timeout logic

[KevLehman]

Proposed changes (including videos or screenshots)

Issue(s)

https://rocketchat.atlassian.net/browse/CTZ-314

Steps to test or reproduce

Further comments

Basically, we were awaiting the response of the first webhook call. This is not bad in circumstances where the webhook is answering like a normal webhook.

If, however, the webhook was not working correctly or couldn't handle the load, we would wait for TIMEOUT before resolving the message being sent or the action being performed.

Also, the webhook was retrying on cases where it shouldn't. Like for example, when it was returning 400~ status codes.

Finally, tests.

[dionisio-bot]

Looks like this PR is not ready to merge, because of the following issues:

• This PR is targeting the wrong base branch. It should target 7.11.0, but it targets 7.10.0

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

⚠️ No Changeset found

Latest commit: f943117

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[codecov]

Codecov Report

❌ Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.34%. Comparing base (078362e) to head (f943117).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #36793      +/-   ##
===========================================
- Coverage    66.35%   66.34%   -0.02%     
===========================================
  Files         3322     3323       +1     
  Lines       111249   111282      +33     
  Branches     21075    21081       +6     
===========================================
+ Hits         73824    73834      +10     
- Misses       34743    34753      +10     
- Partials      2682     2695      +13     

Flag	Coverage Δ
e2e	57.67% <ø> (-0.07%)	⬇️
unit	71.63% <81.25%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[scuciatto]

/backport 7.10.4

[dionisio-bot]

Pull request #37433 added to Project: "Patch 7.10.4"