diff options
| author | Ed Tanous <ed@tanous.net> | 2024-04-07 18:38:44 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2024-04-09 01:46:24 +0300 |
| commit | 6ea9076048028a4adfdbdd2606698bffc81203d3 (patch) | |
| tree | 23ba9504c963534c9354e66cdd61b19a5608eafd /http/http_client.hpp | |
| parent | 8cb2c024c4625e2fe2f0b107a865faffcd4bb770 (diff) | |
| download | bmcweb-6ea9076048028a4adfdbdd2606698bffc81203d3.tar.xz | |
Rework logger to create compile time errors
The logger changes to move to std::format incidentally caused format
errors to no longer be flagged at compile time.
The example here[1] shows that the latest gcc/c++ gave us a way to solve
this, using std::format_string.
This incidentally shows two places where we got the format arguments
wrong, so fix them.
[1] https://stackoverflow.com/questions/72795189/how-can-i-wrap-stdformat-with-my-own-template-function
Change-Id: Id884200e2c98eeaf5ef8db6c1d6362ede2ffb858
Signed-off-by: Ed Tanous <ed@tanous.net>
Diffstat (limited to 'http/http_client.hpp')
| -rw-r--r-- | http/http_client.hpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/http/http_client.hpp b/http/http_client.hpp index 2d03487e84..59804789ee 100644 --- a/http/http_client.hpp +++ b/http/http_client.hpp @@ -790,8 +790,7 @@ class ConnectionPool : public std::enable_shared_from_this<ConnectionPool> { // If we can't buffer the request then we should let the // callback handle a 429 Too Many Requests dummy response - BMCWEB_LOG_ERROR("{}:{} request queue full. Dropping request.", - id); + BMCWEB_LOG_ERROR("{} request queue full. Dropping request.", id); Response dummyRes; dummyRes.result(boost::beast::http::status::too_many_requests); resHandler(dummyRes); |