mirror/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-02-28 01:44:20 -07:00
Code Issues Packages Projects Releases Wiki Activity

[http_request] Deduplicate lowercasing in start() overloads

Browse Source 
Have the deprecated std::set overload forward to the vector overload
which already handles lowercasing.
This commit is contained in:
J. Nick Koston
2026-02-16 18:26:12 -06:00
parent 7c36dfa1fc
commit a55abd8e5f
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
esphome/components/http_request/http_request.h
View File

@@ -363,12 +363,8 @@ class HttpRequestComponent : public Component {
std::shared_ptr<HttpContainer> start(const std::string &url, const std::string &method, const std::string &body,
const std::list<Header> &request_headers,
const std::set<std::string> &collect_headers) {
std::vector<std::string> lower_case_collect_headers;
lower_case_collect_headers.reserve(collect_headers.size());
for (const std::string &collect_header : collect_headers) {
lower_case_collect_headers.push_back(str_lower_case(collect_header));
}
return this->perform(url, method, body, request_headers, lower_case_collect_headers);
return this->start(url, method, body, request_headers,
std::vector<std::string>(collect_headers.begin(), collect_headers.end()));
}
std::shared_ptr<HttpContainer> start(const std::string &url, const std::string &method, const std::string &body,