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

[http_request] Rename should_collect_header param to lower_header_name

Browse Source 
Makes it clear the caller must pass an already-lowercased header name.
This commit is contained in:
J. Nick Koston
2026-02-16 18:37:08 -06:00
parent f43ef21c69
commit 7726d0aac9
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -81,9 +81,10 @@ inline bool is_redirect(int const status) {
inline bool is_success(int const status) { return status >= HTTP_STATUS_OK && status < HTTP_STATUS_MULTIPLE_CHOICES; }
/// Check if a header name should be collected (linear scan, fine for small lists)
inline bool should_collect_header(const std::vector<std::string> &collect_headers, const std::string &header_name) {
inline bool should_collect_header(const std::vector<std::string> &collect_headers,
const std::string &lower_header_name) {
for (const auto &h : collect_headers) {
if (h == header_name)
if (h == lower_header_name)
return true;
}
return false;