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

better XMPP integration #28

Open
oxpa opened this issue Feb 3, 2017 · 1 comment
Open

better XMPP integration #28

oxpa opened this issue Feb 3, 2017 · 1 comment

Comments

@oxpa
Copy link

oxpa commented Feb 3, 2017

Current implementation uses single account to send all messages.
I propose changing this behaviour by using multiple connections into a single conference in XMPP with resources matching telegram user names. (e.g. js@c.j.r/username)

So if you have a chat of user1, user2, user3 you should join javascript@conferences.domain.tld three times with nicknames (and from resources) of /user1, /user2, /user3 and send messages from appropriate resource.

On a nick collision add some prefix or suffix.

Later this may be reworked to create a service when the bot can link two chatroom of different networks by a user command.

@ForNeVeR
Copy link
Collaborator

ForNeVeR commented Feb 4, 2017

First of all, I think that the titular better XMPP integration should be done. You propose a viable solution, but currently I'd vote against that particular solution.

@vitalyster told me about that "component" thing (see XEP-0114). It seems that we can make our bot into a component. That way it'll be authoritative for sending presences on behalf of <username>@telegram.example.com, so it'll be possible to send activity presences and messages from Telegram users to any XMPP server.

I think that we should conduct additional research and go that component way instead of inventing poor ad-hoc solutions for the problem.

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

No branches or pull requests

3 participants