ESP32: IDFv5.0 Fix mqtt module

[serg3295]

• This PR is for the dev branch rather than for the release branch.
• This PR is compliant with the other contributing guidelines as well (if not, please describe why).
• I have thoroughly tested my contribution.
• The code changes are reflected in the documentation at docs/*.

The mqtt module does not work with IDFv5.0
I think an additional user_config checks were added in new espressif/esp-mqtt submodule.

Error log:

NodeMCU ESP32 build unspecified powered by Lua 5.3.5 [5.3-int32-singlefp] on IDF v5.0.2

new m: 	userdata: 0x3ffcac84

> m:connect('mqtt://192.168.1.31')
W (131404) mqtt_client: Transport config set, but overridden by scheme from URI: transport = 1, uri scheme = mqtt
W (131404) mqtt_client: Client asked to stop, but was not started
Lua error: 	stdin:1: MQTT library failed to start
stack traceback:
	[C]: in method 'connect'
	stdin:1: in main chunk
	[C]: in ?
	[C]: in ?

> m:connect('192.168.1.31')
E (159984) mqtt_client: No scheme found
E (159984) mqtt_client: Failed to create transport list
Lua error: 	stdin:1: Error starting mqtt client
stack traceback:
	[C]: in method 'connect'
	stdin:1: in main chunk
	[C]: in ?
	[C]: in ?

This PR fixes that error.

[pjsg]

LGTM

[serg3295]

Small fixes.
Select next parameter in mqtt:connect() when port or secure are nil
Test cases:

m:connect('mqtt://192.168.1.31', nil, nil, 0, function()
    print("MQTT connected! Full uri.")
  end,
  function(client)
    print("MQTT disconnected! Full uri.")
  end
)

-- or
m:connect('mqtt://192.168.1.31', nil, 0)

[serg3295]

Sorry, my last commit incorrectly handles the case
mqtt:connect("server", function(client) print("connected") end, handle_mqtt_error)
An additional check on port==nil, secure==nil is required before n++;

Fixed.

[jmattsson]

Looks good to me, though I'm (still) not in a position to try it out myself.

[jmattsson]

Well oops! 😳 That would be my mistake...

[jmattsson]

Hmm, I don't see any reference to these ever having existed in this module. I guess this was an esp8266 leftover?

[serg3295]

Yes, I found these reason codes only in the mqtt-8266 module.

[jmattsson]

Thank you for the PR @serg3295 ! Merge has been done!