Mqtt connection timeout setting

RE: Mqtt connection timeout setting

Håkon Alseth — Mon, 02 Aug 2021 07:32:46 GMT

Hi,

My apologies for the late reply.

[quote user="Stratosphere"]I am currently using sdk 1.4.2. should this work on it?[/quote]

This functionality was not added until more recent. Is it technically possible for you to use a newer ncs version?

Kind regards,

Håkon

RE: Mqtt connection timeout setting

Stratosphere — Sun, 18 Jul 2021 08:20:49 GMT

Hi Haken,

I am currently using sdk 1.4.2. should this work on it?

It does not seem to be working for me. here is the console log

RRC mode: Connected
getaddrInfo errno: 0
fcntl nonblocking set: 0. fd : 0, flags: 16384, rflags: 16384
In http timer. http state: 3, currentState: 0, httpQuit: 0
In http timer. http state: 3, currentState: 3, httpQuit: 0
http waiting to connect
http: Connected

"in http timer" is printed every 5 second by a timer callback. "http waiting to connect" is printed from within the do while loop as per code below:

flags = fcntl(fd, F_GETFL, 0);
ret = fcntl(fd, F_SETFL, flags | O_NONBLOCK);
rflags = fcntl(fd, F_GETFL, 0);	
printk("fcntl nonblocking set: %d. fd : %d, flags: %d, rflags: %d\n", ret, fd, flags, rflags);

fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TLS_1_2);
if (fd == -1) 
{
	if (DEBUG_PRINT_HTTP) printk("Failed to open socket!\n");
	errorReason = ERROR_HTTP_SOCKET_FAILED;
	goto clean_up;
}

httpState = 2;
/* Setup TLS socket options */
err = httpTlsSetup(fd);
if (err) 
{
	errorReason = ERROR_HTTP_SECURECLIENT_INIT_FAIL;
	goto clean_up;
}

httpState = 3;

//Connect
{
	waitingToConnect = 0;
	do 
	{
		err = connect(fd, res->ai_addr , sizeof(struct sockaddr_in));
		k_sleep(100);
		waitingToConnect++;
		printk("http waiting to connect\n");
	} while ((err == EINPROGRESS) && (waitingToConnect <300));

	if (err || waitingToConnect >=300)
	{
		if (DEBUG_PRINT_HTTP) printk("connect() failed, time: %d, err: %d\n", waitingToConnect, errno);
		if (errno == 23)
			sys_reboot(0);

		errorReason = ERROR_HTTP_CONNECT_FAILED;
		goto clean_up;
	}
	if (DEBUG_PRINT_HTTP) printk("http: Connected\n");
}

"http waiting to connect" should be printed every 100ms if connect() is non-blocking. so it seems that connect() is still using a blocking call.

RE: Mqtt connection timeout setting

Håkon Alseth — Tue, 06 Jul 2021 11:24:15 GMT

Hi,

That is up to you how you time out, you can do it this way by breaking out of the loop (and handling the closing scenario), but if the connect() succeeds; you need to handle the scenario of having non-blocking socket operations going on forward (or go back to blocking)

Kind regards,

Håkon

RE: Mqtt connection timeout setting

Stratosphere — Mon, 05 Jul 2021 16:08:44 GMT

Thank you so much Haken.

Will test this out and let you know.\

so basically, the do-while loop will keep calling connect repetitively until EINPROGRESS is being returned.
If I wanted to exit, say after 20 seconds, i can just break out of the while loop and go to clean up? Will that crash anything when the connect request finally returns a few minutes later but the socket has been closed?

RE: Mqtt connection timeout setting

Håkon Alseth — Mon, 05 Jul 2021 09:22:20 GMT

Hi,

If you enable non-blocking socket operations, the connect() call will return EINPROGRESS until its done.

That can be done like this:

#include <fcntl.h>

    ....
	printk("enable non-blocking connect\n");
	printk("Will return EINPROGRESS (%d) when in progress\n", EINPROGRESS);
	int flags = fcntl(fd, F_GETFL, 0);
	int ret = fcntl(fd, F_SETFL, flags | O_NONBLOCK);
	int rflags = fcntl(fd, F_GETFL, 0);	
	
	
	printk("connect time: %lld\n", k_uptime_get());
	ret = connect(...)
	printk("connect time: %lld\n", k_uptime_get());
	....

I added the logic to the https_client sample in ncs v1.6.0 just so that you can see that the call returns almost immediately when you enable nonblocking socket operation.

devzone.nordicsemi.com/.../nonblock_5F00_https_5F00_client_5F00_ncs_5F00_v1.6.0.diff

Kind regards,

Håkon

RE: Mqtt connection timeout setting

Stratosphere — Fri, 02 Jul 2021 22:43:40 GMT

thank you Haken.

Can you help me add a timeout for http connection requests?

Regards

Noaman

RE: Mqtt connection timeout setting

Håkon Alseth — Thu, 01 Jul 2021 14:05:15 GMT

Hi,

Unfortunately, there is no option to set the timeout at this time, but I will add this as a feature request internally.

Kind regards,

Håkon