Lack of handling on _requires_ack if this.store throws error leads to infinite this._message_identifier increment

[elcreator]

paho.mqtt.javascript/src/paho-mqtt.js

Line 1201 in be026e0

if (this._message_identifier === this.maxMessageIdentifier) {

should be

 if (this._message_identifier >= this.maxMessageIdentifier) { 

because

			while(this._sentMessages[this._message_identifier] !== undefined) {
				this._message_identifier++;
			}

increments before this.store("Sent:", wireMessage); is called and this.store("Sent:", wireMessage); can throw an error, after which this._message_identifier remains incremented. Next time when this._requires_ack will be called it will be incremented again and this comparison will not work!