eclipse-paho / eclipse-paho/paho.mqtt.java

MQTTv5 the mqttErrorOccured (sic) callback does not work

Open
#537 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
2.3k
Forks
919
PR merge metrics
No merged PRs in 30d

Description

The purpose of the mqttErrorOccured callback is to allow the client application to decide whether to accept various errors in the input stream and to continue or close the connection. However, this does not work for three reasons:

1. The current paho code does not check most conditions where the input stream violates the spec.
2. The current paho code does not call this callback in most cases it does detect problems
3. In the one case it does call this callback (invalid topic alias) the connection is closed regardless of what the callback does.

Note: In both US and UK English the correct spelling of occurred has two c's and two r's and therefore the method mqttErrorOccured is misspelled.

- [ ] Bug exists Release Version 1.2.0 ( Master Branch)
- [ ] Bug exists in MQTTv3 Client on Snapshot Version 1.2.1-SNAPSHOT (Develop Branch)
- [x] Bug exists in MQTTv5 Client on Snapshot Version 1.2.1-SNAPSHOT (Develop Branch)

If this is a bug regarding the Android Service, please raise the bug here instead: https://github.com/eclipse/paho.mqtt.android/issues/new

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.