PublishWithOptions() will not return ErrInvalidArguments when QoS is invalid

[Vishnu Reddy]

paho.golang/paho/client.go

Line 885 in ef0065f

 return nil, fmt.Errorf("QoS isn't 0, 1 or 2") 

This should have ErrInvalidArguments in its tree so the application knows it is a fatal error. This is a very quick fix and I am happy to open a PR for it.

[Matt Brittan]

This is a very quick fix and I am happy to open a PR for it.

Please do - keen to get a release out this week and would be nice to have this fixed.

[Vishnu Reddy]

Will open a PR today or tomorrow (US Eastern time). Would you like me to incorporate a fix to #233 as well? @MattBrittan

[Matt Brittan]

Would you like me to incorporate a fix to #233 as well

Yes please.