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

Added possibility to QXmppServer to route a stanza to a specific resource #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

servonic
Copy link
Contributor

Added possibility to QXmppServer to route a stanza to a specific resource without the need to add the resource to the stanza's JID. Closes #7

@servonic
Copy link
Contributor Author

I updated the pull request (I hope I understood you correctly):
I moved the if into routeData (passing the resource through sendElement / sendPacket) and prevent routing if "to" is already a full JID.
Please let me know whether you want me to sqash it into a single commit, or if you still have concerns whether you have a better idea to solve this.

@servonic
Copy link
Contributor Author

servonic commented Jan 5, 2016

@jlaine I rebased (and tested again) the changed I did to the pull request. Unfortunately the rebasing seems to have deleted our comments - sorry for that!
I think I moved the decision to the point you wanted it.
Can you please recheck and if you still see problems, please let me know what I should change.

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

Successfully merging this pull request may close these issues.

QXmppServer: Route stanza for a bare jid to specific resource
1 participant