ESP8266 WiFi gateway issue

hek

Could you try replacing this in MySensors/core/MyGatewayTransportEthernet.cpp

bool _readFromClient(uint8_t i) {
	while (clients[i].connected() && clients[i].available()) {
		char inChar = clients[i].read();
		if (inputString[i].idx < MY_GATEWAY_MAX_RECEIVE_LENGTH - 1) {
			// if newline then command is complete
			if (inChar == '\n') {
				// a command was issued by the client
				// we will now try to send it to the actuator
				// echo the string to the serial port
				debug(PSTR("Client %d: %s\n"), i, inputString[i].string);

				// add string terminator and clear the string:
				inputString[i].string[inputString[i].idx] = 0;
				inputString[i].idx = 0;
				bool ret = protocolParse(_ethernetMsg, inputString[i].string);
				if (ret) {
					return ret;
				}

			} else {
				// add it to the inputString:
				inputString[i].string[inputString[i].idx++] = inChar;
			}
		} else {
			// Incoming message too long. Throw away
			debug(PSTR("Client %d: Message too long\n"), i);
			inputString[i].idx = 0;
			// Finished with this client's message. Next loop() we'll see if there's more to read.
			break;
		}
	}
	return false;
}

And let me know how it behaves.

GizMoCuz

I think that will solve the issue... @snow, waiting patiently for your reply ;)

snow

It's much better now. I can control dimmer from the domoticz. :)

0;0;3;0;9;Client 0 connected
0;0;3;0;9;Client 0: 0;0;3;0;2;Get Version
0;0;3;0;2;Get Version
0;0;3;0;9;read: 2-2-0 s=255,c=0,t=17,pt=0,l=10,sg=0:1.6.0-beta
0;0;3;0;9;read: 2-2-0 s=255,c=3,t=6,pt=1,l=1,sg=0:0
0;0;3;0;9;Client 0: 2;255;3;0;6;M Version
2;255;3;0;6;M
0;0;3;0;9;send: 0-0-2-2 s=255,c=3,t=6,pt=0,l=1,sg=0,st=ok:M
0;0;3;0;9;read: 2-2-0 s=1,c=0,t=4,pt=0,l=0,sg=0:
0;0;3;0;9;read: 2-2-0 s=255,c=3,t=11,pt=0,l=21,sg=0:DimmableLED /w bu
0;0;3;0;9;read: 2-2-0 s=255,c=3,t=12,pt=0,l=3,sg=0:1.2
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=1,l=1,sg=0:100
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=1,l=1,sg=0,st=ok:100
0;0;3;0;9;Client 0: 2;1;1;1;3;93M
2;1;1;1;3;93
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:93
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:93
0;0;3;0;9;Client 0: 2;1;1;1;3;86
2;1;1;1;3;86
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:86
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:86
0;0;3;0;9;Client 0: 2;1;1;1;3;79
2;1;1;1;3;79
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:79
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:79
0;0;3;0;9;Client 0: 2;1;1;1;3;71
2;1;1;1;3;71
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:71
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:71
0;0;3;0;9;Client 0: 2;1;1;1;3;64
2;1;1;1;3;64
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:64
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:64
0;0;3;0;9;Client 0: 2;1;1;1;3;57
2;1;1;1;3;57
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:57
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:57
0;0;3;0;9;Client 0: 2;1;1;1;3;50
2;1;1;1;3;50
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:50
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:50
0;0;3;0;9;Client 0: 2;1;1;1;3;43
2;1;1;1;3;43
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:43
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:43
0;0;3;0;9;Client 0: 2;1;1;1;3;36
2;1;1;1;3;36
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:36
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:36
0;0;3;0;9;Client 0: 2;1;1;1;3;36
2;1;1;1;3;36
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:36
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:36
0;0;3;0;9;Client 0: 2;1;1;1;3;29
2;1;1;1;3;29
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:29
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:29
0;0;3;0;9;Client 0: 2;1;1;1;3;21
2;1;1;1;3;21
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:21
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:21
0;0;3;0;9;Client 0: 2;1;1;1;3;14
2;1;1;1;3;14
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=2,sg=0,st=ok:14
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=2,sg=0:14
0;0;3;0;9;Client 0: 2;1;1;1;3;74
2;1;1;1;3;7
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=0,l=1,sg=0,st=ok:7
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=0,l=1,sg=0:7
0;0;3;0;9;Client 0: 2;1;1;1;2;0
2;1;1;1;2;0
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=2,pt=0,l=1,sg=0,st=ok:0
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=2,pt=0,l=1,sg=0:0
0;0;3;0;9;read: 2-2-0 s=1,c=1,t=3,pt=2,l=2,sg=0:0
0;0;3;0;9;send: 0-0-2-2 s=1,c=1,t=3,pt=2,l=2,sg=0,st=ok:0

hek

Great, I'll push this fix ASAP.

Hoffan

@Hoffan said:
anyone that knows if include button working now? With vera as controller

Hoffan

I can answer that question by my self.. Its still not working

hek

@Hoffan, I verified the inclusion button functionality in the dev branch during the weekend (at least on AtMega but it should work fine on ESP as well).

Just make sure to enable the feature and set a valid ESP-pin. All is documented in sketch.

Hoffan

@hek

Okey So i must enable it in the sketch even if i only use the start button in the GUI in Vera

hek

@Hoffan
Not the "button" feature (unless you use it), but the "inclusion mode" feature must be enabled.

Hoffan

I don't know if i understan you right here, @Hek

if i like to use the start button in vera should it look like this

// Enable inclusion mode
define MY_INCLUSION_MODE_FEATURE
// Enable Inclusion mode button on gateway
#define MY_INCLUSION_BUTTON_FEATURE
// Set inclusion mode duration (in seconds)
define MY_INCLUSION_MODE_DURATION 60
// Digital pin used for inclusion mode button
#define MY_INCLUSION_MODE_BUTTON_PIN 3

hek

You only need these two:

#define MY_INCLUSION_MODE_FEATURE
#define MY_INCLUSION_MODE_DURATION 60

Hoffan

Yes That i understand, But Nothing is still happening in the GUI men i press start

hek

Anything in the gateway log?

Hoffan

hek

Hmm. ok so message reaches gateway. Good start. It should reply back to Vera. with a
0;0;3;0;5;1

Unfortunately there is no debug log for this in the code. So I can't see if the message is sent to your controller. So either you'll have to look in the Vera log or add a

Serial.println(_ethernetMessage);

Here: https://github.com/mysensors/Arduino/blob/development/libraries/MySensors/core/MyGatewayTransportEthernet.cpp#L170

To see that the message is actually sent back to vera.

Hoffan

You mean that i should change that line #170 år just put in the serial string

hek

Add the string debug print.

ESP8266 WiFi gateway issue

10

11.7k

11.2k

113.1k