Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Courier.java line 112 subscribes to malformed topic #3

Open
Garepjotr opened this issue Mar 3, 2013 · 2 comments
Open

Courier.java line 112 subscribes to malformed topic #3

Garepjotr opened this issue Mar 3, 2013 · 2 comments
Assignees

Comments

@Garepjotr
Copy link

public void subm(String topicname){
try {
this.subscribe(this.topic+ "/input/"+topicname);

Do this instead:
this.subscribe(topicname);

@ghost ghost assigned ajtag Mar 5, 2013
@ajtag
Copy link
Owner

ajtag commented Mar 5, 2013

Not sure I agree with this one.
The reason for having this.topic+'input' is to seperate Input from Ouput notifications and so multiple servers can exist within the same MQTT server.

topicname is effectively the third line from a sign, so this leaves the user to stop name colisions

@Garepjotr
Copy link
Author

but you do
subm(topic+"/input/#");
wich would subscibe you to:
minecraft/input/minecraft/input/#

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants