Not authorized MQTT connection related to //#define MQTT_DEBUG

[davidjohngraham]

• Arduino board: UNO R3 SMD and FONA800
• Arduino IDE version (found in Arduino -> About Arduino menu): 1.6.9 (programming via Visual Studio/vMicro)
• List the steps to reproduce the problem below (if possible attach a sketch or
  copy the sketch code in too):

Attempting to connect to io.adafruit.com via MQTT on port 1883 using a FONA 800. Receive "Not authorized to connect" response.

In attempting to find the source of this problem I uncommented #define MQTT_DEBUG in line 37 of Adafruit_MQTT.h. I was then able to connect. Recommenting it prevented connection, so I am confident this is the source of the problem.

Please would you investigate and fix this problem? I'm afraid my c++ skills are not up to the job.

Thanks

[spa-sam]

I confirm the problem exists! lib 0.16.0 -1

[Maddox-zephyr]

Well I confirm that this bug still exists - it has been driving me crazy as I made a minor modification to a working sketch and started getting the "not authorized" error. I uncommented the MQTT_DEBUG as suggested by davidjohngraham and now it connects.

[carson-51]

I confirm the problem exists. I noticed that if I use the MQTT FONA example, it works okay, but when I switch to my sketch (built off the example) it has the "Not authorized to connect" issue. I also notice that sometimes it states a more generic "Connection fail" error. After repeatedly re-uploading the same sketch it seems to go away and work properly for a while. I've tried rebooting the MQTT broker (Windows 7 computer), but it doesn't seem to make any difference. Once I uncommented line 37 "MQTT_DEBUG" as suggested, it works.

[golfredom]

It is happening to me. thanks to this post , I quite solve it. Is it repaired already, or are we still waiting for an update?
best

[bakercp]

Confirming this error and workaround.

[revop]

Hi, I made the workaround but it did not work:

I am still having the same error:
Connected to Cellular!
Connecting to TCP
---> AT+CIPSHUT
<--- SHUT OK
---> AT+CIPMUX=0
<--- OK
---> AT+CIPRXGET=1
<--- OK
AT+CIPSTART="TCP","io.adafruit.com","1883"
<--- OK
<--- CONNECT OK
MQTT connect packet:
[0x10], 6 [0x36], [0x00], [0x04], M [0x4D], Q [0x51], T [0x54], T [0x54],
[0x04], [0xC2], [0x01], , [0x2C], [0x00], [0x00], [0x00], [0x06],
r [0x72], e [0x65], v [0x76], o [0x6F], p [0x70], e [0x65], [0x00], [0x20],
0 [0x30], 7 [0x37], 1 [0x31], e [0x65], 0 [0x30], 4 [0x34], 0 [0x30], a [0x61],
6 [0x36], 3 [0x33], 5 [0x35], 1 [0x31], 4 [0x34], 5 [0x35], 5 [0x35], 1 [0x31],
a [0x61], 2 [0x32], a [0x61], 5 [0x35], 0 [0x30], 9 [0x39], c [0x63], 7 [0x37],
c [0x63], 0 [0x30], 5 [0x35], 7 [0x37], 1 [0x31], 4 [0x34], 6 [0x36], 8 [0x38],

Writing packet
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
AT+CIPSEND=56
0x10 0x36 0x0 0x4 0x4D 0x51 0x54 0x54 0x4 0xFFFFFFC2 0x1 0x2C 0x0 0x0 0x0 0x6 0x72 0x65 0x76 0x6F 0x70 0x65 0x0 0x20 0x30 0x37 0x31 0x65 0x30 0x34 0x30 0x61 0x36 0x33 0x35 0x31 0x34 0x35 0x35 0x31 0x61 0x32 0x61 0x35 0x30 0x39 0x63 0x37 0x63 0x30 0x35 0x37 0x31 0x34 0x36 0x38
<--- >
<--- SEND OK
Reading data..
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
---> AT+CIPRXGET=4
<--- +CIPRXGET: 4,4
4 bytes available
1 bytes read
0x20
Read: [0x20],
Packet Type: [0x20],
Reading data..
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
---> AT+CIPRXGET=4
<--- +CIPRXGET: 4,3
3 bytes available
1 bytes read
0x2
Read: [0x02],
Packet Length: 2
Reading data..
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
---> AT+CIPSTATUS
<--- OK
<--- STATE: CONNECT OK
---> AT+CIPRXGET=4
<--- +CIPRXGET: 4,2
2 bytes available
2 bytes read
0x0 0x5
Read: [0x00], [0x05],
Not authorized to connect
MQTT connection failed, resetting...

Can someone help me?

[depperson]

I'm on Arduino Ethernet and had the same problem. Enabling MQTT_DEBUG also worked for me.

[saucepwn]

I have an Arduino Uno with the generic Ethernet shield, I can confirm it's broken unless I enable MQTT_DEBUG.

[R0dinger]

Same problem here 2017-12-05.
Using Arduino Mega with Ethernet shield.
Uncommented #define MQTT_DEBUG in Adafruit_MQTT.h and it works fine now.

[king-jam]

Encountered the same issue. Still trying to debug but preliminary debug shows that it is a constructor issue. There are so many constructor overrides, I think the compiler is getting confused as to which to use and therefore the MQTT connect packet doesn't include the AIO_KEY parameter.

This theory seems evident in https://github.com/adafruit/Adafruit_MQTT_Library/blob/master/Adafruit_MQTT_FONA.h#L44

Given the FONA example everyone uses has 5 params: https://github.com/adafruit/Adafruit_MQTT_Library/blob/master/examples/mqtt_fona/mqtt_fona.ino#L57

I believe it is defaulting the key to empty as empty is the last parameter....just a theory, I'm not a great C++ guy.

[borch84]

I confirm that I am using version 0.17 of the Adafruit MQTT library and I didn't have to use the MQTT_DEBUG option when connecting with username/password. I am testing this library against a mosquitto broker running on a Raspberry Pi3

[king-jam]

Follow up to: #54 (comment)

I confirmed in my builds that when I had an issue, it was seeing the AIO_KEY parameter as empty. This appears to be a compilation issue that I resolved when I changed the ordering of input parameters in this file: https://github.com/adafruit/Adafruit_MQTT_Library/blob/master/Adafruit_MQTT_FONA.h

[denizruzgar]

@king-jam
I have the same issue. Uncommenting debug or your method to change the ordering of the inputs don't work for me. Do you have any other options in mind ?

[denizruzgar]

I received a notification from Arduino IDE to update my MQTT library 30 mins ago to v 0.20. Now my code which used to connect to MQTT, is going into a loop and restarts the whole demo for some reason.

[king-jam]

@denizruzgar what happens if you just delete these lines?

https://github.com/adafruit/Adafruit_MQTT_Library/blob/master/Adafruit_MQTT_FONA.h#L44-L48