ConnectProperties in cp_connect.go has a MaximumQOS field that is not part of the MQTT 5 specification

[Vishnu Reddy]

The MQTT 5 spec doesn't include a MaximumQOS field in CONNECT properties, but it is included as a field.

paho.golang/paho/cp_connect.go

Line 29 in 21b4033

 MaximumQOS            *byte 

[Matt Brittan]

This ends up as property 36 "Maximum QoS" but that is only valid on CONNACK packets so this looks like a bug.

[Matt Brittan]

This also needs to be fixed in properties.go -

PropMaximumQOS: {CONNECT: {}, CONNACK: {}},

should be:

PropMaximumQOS: {CONNACK: {}},