diff --git a/platformio.ini b/platformio.ini index 533cd25e0..a742abab0 100644 --- a/platformio.ini +++ b/platformio.ini @@ -39,7 +39,7 @@ unbuild_flags = [espressi32_base] ; 6.7.0 = Arduino v2.0.16 (based on IDF v4.4.7). See https://github.com/platformio/platform-espressif32/releases/tag/v6.7.0 -platform = espressif32@6.8.0 +platform = espressif32@6.8.1 framework = arduino board_build.filesystem = littlefs build_flags = diff --git a/src/console.cpp b/src/console.cpp index 33113a2a1..3d4dfb0c6 100644 --- a/src/console.cpp +++ b/src/console.cpp @@ -541,14 +541,14 @@ static void setup_commands(std::shared_ptr & commands) { cmd = Command::parse_command_string(cmd, id); // extract hc or dhw } if (cmd == nullptr) { - cmd = device_type == EMSdevice::DeviceType::SYSTEM ? F_(info) : F_(values); + cmd = F_(values); } if (arguments.size() == 2) { // no value specified, just the cmd return_code = Command::call(device_type, cmd, nullptr, true, id, json); } else if (arguments.size() == 3) { - if ((strncmp(cmd, F_(info), 4) == 0) || strncmp(cmd, F_(values), 6) == 0) { + if (!strcmp(cmd, F_(info)) || !strcmp(cmd, F_(values))) { // info has a id but no value return_code = Command::call(device_type, cmd, nullptr, true, atoi(arguments.back().c_str()), json); } else if (arguments[2] == "?") { diff --git a/src/web/WebSchedulerService.cpp b/src/web/WebSchedulerService.cpp index 48d17f1e2..de72da031 100644 --- a/src/web/WebSchedulerService.cpp +++ b/src/web/WebSchedulerService.cpp @@ -334,31 +334,32 @@ bool WebSchedulerService::command(const char * name, const std::string & command // parse json JsonDocument doc; if (deserializeJson(doc, cmd) == DeserializationError::Ok) { - HTTPClient http; - int httpResult = 0; - String url = doc["url"] | ""; + HTTPClient http; + int httpResult = 0; + std::string url = doc["url"] | ""; // for a GET with parameters replace commands with values - auto q = url.indexOf('?'); - if (q != -1) { - auto s = url.substring(q + 1); - std::string v = s.c_str(); - commands(v, false); - url.replace(s, v.c_str()); + // don't search the complete url, it may contain a devicename in path + auto q = url.find_first_of('?'); + if (q != std::string::npos) { + auto s = url.substr(q + 1); // copy only parameters + auto l = s.length(); + commands(s, false); + url.replace(q + 1, l, s); } - if (url.startsWith("http") && http.begin(url)) { + if (!url.find("http") && http.begin(url.c_str())) { // add any given headers for (JsonPair p : doc["header"].as()) { http.addHeader(p.key().c_str(), p.value().as().c_str()); } - String value = doc["value"] | data.c_str(); // extract value if its in the command, or take the data - String method = doc["method"] | "GET"; // default GET + std::string value = doc["value"] | data.c_str(); // extract value if its in the command, or take the data + std::string method = doc["method"] | "GET"; // default GET // if there is data, force a POST if (value.length() || method == "post") { // we have all lowercase - if (value.startsWith("{")) { + if (value.find_first_of('{') != std::string::npos) { http.addHeader("Content-Type", "application/json"); // auto-set to JSON } - httpResult = http.POST(value); + httpResult = http.POST(value.c_str()); } else { httpResult = http.GET(); // normal GET } diff --git a/src/web/shuntingYard.hpp b/src/web/shuntingYard.hpp index aa19ec813..46f5b2174 100644 --- a/src/web/shuntingYard.hpp +++ b/src/web/shuntingYard.hpp @@ -612,29 +612,29 @@ std::string compute(const std::string & expr) { std::string cmd = expr_new.substr(f, e - f).c_str(); JsonDocument doc; if (DeserializationError::Ok == deserializeJson(doc, cmd)) { - HTTPClient http; - String url = doc["url"] | ""; - if (url.startsWith("http") && http.begin(url)) { + HTTPClient http; + std::string url = doc["url"] | ""; + if (!url.find("http") && http.begin(url.c_str())) { int httpResult = 0; for (JsonPair p : doc["header"].as()) { http.addHeader(p.key().c_str(), p.value().as().c_str()); } - String value = doc["value"] | ""; - String method = doc["method"] | "GET"; // default GET + std::string value = doc["value"] | ""; + std::string method = doc["method"] | "GET"; // default GET // if there is data, force a POST if (value.length() || method == "post") { - if (value.startsWith("{")) { + if (value.find_first_of('{') != std::string::npos) { http.addHeader("Content-Type", "application/json"); // auto-set to JSON } - httpResult = http.POST(value); + httpResult = http.POST(value.c_str()); } else { httpResult = http.GET(); // normal GET } if (httpResult > 0) { std::string result = emsesp::Helpers::toLower(http.getString().c_str()); - String key = doc["key"] | ""; + std::string key = doc["key"] | ""; doc.clear(); if (key.length() && DeserializationError::Ok == deserializeJson(doc, result)) { result = doc[key.c_str()].as(); @@ -644,7 +644,7 @@ std::string compute(const std::string & expr) { http.end(); } } - f = expr_new.find_first_of("{", e); + f = expr_new.find_first_of('{', e); } // positions: q-questionmark, c-colon