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

Improve MQTTv5 mqttErrorOccured behaviour.

Open
#590 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

Please fill out the form below before submitting, thank you!

- [ ] 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

The basic mechanism for the mqttErrorOccured does not work. In the one and only place it is called (Invalid Topic Alias) whatever the user decides an MQTT exception is thrown on return. None of the various protocol errors are thrown.

Contributor guide

Open the contributing guide

Research direction

Start by locating the MQTTv5 mqttErrorOccured callback and the Invalid Topic Alias handling mentioned in the report. Trace how protocol errors are propagated to determine the intended callback behavior; done should mean MQTTv5 protocol errors reach the callback without an unintended exception being thrown.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
networking
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.