For requests to be processed by ThingSpeak.com, MustCheckReply will need to be set to True.
I am a developer working for ThingSpeak. A permanent change to our system architecture has the unintended consequence of ignoring requests from clients which close the TCP connection immediately following the HTTP GET or POST. In order to have requests properly processed by ThingSpeak, the client needs to wait for a response. We have confirmed this in our testing. The change will go live the first week of Feb-2019.
So that change is going live tomorrow?
And is it enough to just document this setting described in the subject?
Or do we need to change something else too?
Feb 4, 2019 is the target date for the change on ThingSpeak.
We did a test run for a few hours a month ago and we saw bunch of letscontrolit requests failing. I'll be honest, I'm not familiar enough with ESPEasy code to change it myself and request a merge. But from spending an hour looking at the code I can see that MustCheckReply needs to be set to true in order for it work going forward. I'm not sure if you do that at the _C004.ino level or something higher.
My hope was to alert you of the upcoming problem before it hit.
See also replies here:
https://github.com/letscontrolit/ESPEasy/issues/2289#issuecomment-462188764
We need to have the settings default to acknowledge reply and then we can close this right?
I guess so
I believe this issue (MustCheckReply=True) is affecting sendToHTTP. Please see the long discussion at https://www.letscontrolit.com/forum/viewtopic.php?f=6&t=6954.
FWIW, sendToHTTP uses MustCheckReply=false. This is hard coded in the function.
Add this information to the wiki page, please. It was difficult to find it.
Add this information to the wiki page, please. It was difficult to find it.
You mean for the sendToHttp command?
That check reply option is less than a week old.
For the controllers, it is already present in the Read The Docs but I do admit it may need its own chapter.
I mean settings for thingspeak. Check Reply: check Acknowledgment
Do you have a link of the docs where it should be added, or at least where things are unclear?
Ah the wiki... well we plan on move the documentation to the Read The Docs, so the wiki has not been maintained for a long time now.
Most helpful comment
Feb 4, 2019 is the target date for the change on ThingSpeak.
We did a test run for a few hours a month ago and we saw bunch of letscontrolit requests failing. I'll be honest, I'm not familiar enough with ESPEasy code to change it myself and request a merge. But from spending an hour looking at the code I can see that MustCheckReply needs to be set to true in order for it work going forward. I'm not sure if you do that at the _C004.ino level or something higher.
My hope was to alert you of the upcoming problem before it hit.