From e65f6c7bc75567c4f5c3750879e6b1fc50e2b88a Mon Sep 17 00:00:00 2001
From: Will Miles <will@willmiles.net>
Date: Sun, 10 Nov 2024 18:34:20 -0500
Subject: [PATCH] handleHttpApi: Remove 'IN' tag

Internal requests are indicated by passing nullptr for the request
argument.  If we *are* in fact called from an HTTP request, we must
generate some kind of reply anyways, so this parameter is obsolete.
---
 wled00/set.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/wled00/set.cpp b/wled00/set.cpp
index 8d2024fa3..23155c34d 100644
--- a/wled00/set.cpp
+++ b/wled00/set.cpp
@@ -1188,8 +1188,7 @@ bool handleHttpApi(AsyncWebServerRequest *request, const String& req, bool apply
   stateUpdated((pos >= 0) ? CALL_MODE_NO_NOTIFY : CALL_MODE_DIRECT_CHANGE);
 
   // internal call, does not send XML response
-  pos = req.indexOf(F("IN"));
-  if ((request != nullptr) && (pos >= 0)) {
+  if (request != nullptr) {
     auto response = request->beginResponseStream("text/xml");
     XML_response(*response);
     request->send(response);