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

Add testing usernames with UTF-8 and XEP-0106 escaping #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fenek
Copy link
Member

@fenek fenek commented Nov 12, 2013

No description provided.

@michalwski
Copy link
Contributor

I know it's quite an old pull request now but I'd like to get back to it. I think better option would be adding a new user with some special characters in nick name and using this user in some new test suite with the same logic but different ways of connection:

  • the user is connected via regular TCP connection
  • connected via BOSH
  • and connected via WS

Current situation where all users have changed usernames can make debugging more difficult for human :)

@michalwski
Copy link
Contributor

@fenek what you think about my prev comment?

@fenek
Copy link
Member Author

fenek commented Jun 6, 2014

I think you're right. I'll change this when I'll have few spare hours.

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.

2 participants