feat: Add request_timeout, a parameter as optional to `cluster::request()` which allows user to set `http_client::timeout` and add error message for request timeouts

sony-string commented 2 weeks ago

At now, cluster::request can't accept HTTP requests that require a long time to wait for response, especially which takes over 5 seconds. That's because there's no option to set http_client::timeout as user want. So, I added the optional parameter request_timeout to set it.

When an user calls cluster::request(), It would be passed through cluster::request() -> http_request::request_timeout (what I added in this commit) -> http_client::timeout

And also, http_request::run is printing Malformed HTTP response when request has timed out, so I fix it to print a log "Timed out while waiting for the response". For this, I needed to add a public member variable to http_client, timed_out which would be set by https_client::one_second_timer()

Code change checklist

[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[x] I tested that my change works before raising the PR.
[x] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

CLAassistant commented 2 weeks ago

All committers have signed the CLA.

netlify[bot] commented 2 weeks ago

Deploy Preview for dpp-dev ready!

Name	Link
Latest commit	b61522d3eeae144a9d3539de2d0bdeff8e9ce9f3
Latest deploy log	https://app.netlify.com/sites/dpp-dev/deploys/668541dcadc549000892f7ff
Deploy Preview	https://deploy-preview-1186--dpp-dev.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

brainboxdotcc / DPP