anna/IM-server-new
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Changed SUBACK response to accept all requested QoS on SUBSCRIBE request

Browse Source
This commit is contained in:
andrea
2013-12-27 16:45:25 +01:00
parent a6f94cb0a3
commit 3f0724d1dc
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
broker/src/main/java/org/dna/mqtt/moquette/messaging/spi/impl/ProtocolProcessor.java
+5 -3
View File
@@ -457,10 +457,12 @@ class ProtocolProcessor implements EventHandler<ValueEvent> {
SubAckMessage ackMessage = new SubAckMessage(); SubAckMessage ackMessage = new SubAckMessage();
ackMessage.setMessageID(msg.getMessageID()); ackMessage.setMessageID(msg.getMessageID());
//TODO by now it handles only QoS 0 messages //reply with requested qos
for (int i = 0; i < msg.subscriptions().size(); i++) { for(SubscribeMessage.Couple req : msg.subscriptions()) {
ackMessage.addType(AbstractMessage.QOSType.MOST_ONE); AbstractMessage.QOSType qos = AbstractMessage.QOSType.values()[req.getQos()];
ackMessage.addType(qos);
} }
LOG.debug("replying with SubAck to MSG ID {}", msg.getMessageID()); LOG.debug("replying with SubAck to MSG ID {}", msg.getMessageID());
session.write(ackMessage); session.write(ackMessage);
} }