Multiple subscription to topics

[abhaytambe]

Looks like the client fails to subscribe successfully to multiple topics. We receive a 1002 error code back on subscribing to the second topic.
i.e. Let's say, we have two topics to subscribe to, 'Topic ABC' and 'Topic DEF'. The client subscribes to topic 'Topic ABC' successfully, but we get a 1002 error back and the connection closes as soon as I subscribe to 'Topic DEF'. If we reverse the order of subscription, then the client subscribes to topic 'Topic DEF' successfully but fails while subscribing to second topic 'Topic ABC'.

This clearly confirms that there is an issue while subscribing to more than one topics

[abhaytambe]

The issue lies in the 'StompClientLib' class, subscribe to destination header should have 'StompCommands.commandHeaderDestinationId' header with a unique value per subscription.
Please refer to the documentation from STOMP
https://stomp.github.io/stomp-specification-1.2.html#SUBSCRIBE

[WrathChaos]

Hello @abhaytambe I'm going to solve it in a week, thank you so much for feedback :) Also, remember that always welcome to PR :)

[deeksha-Goyal]

Hi WrathChaos!
Line No: 372 is creating issue.
let headers = [StompCommands.commandHeaderDestination: destination, StompCommands.commandHeaderAck: ack, StompCommands.commandHeaderDestinationId: ""]

I changed above mentioned line in StompClientLib.swift file to this :
let headers = [StompCommands.commandHeaderDestination: destination, StompCommands.commandHeaderAck: ack, StompCommands.commandHeaderDestinationId: destination]
This solved the issue for me. I hope it is correct logically also. Could you please verify it.

[WrathChaos]

Hello @abhaytambe,
I've updated the lib with your logic. Thank you so much for this contribution. Please try the new version : 1.2.4
Have fun!