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

Support for split XMPP/web domains #69

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

mwild1
Copy link
Member

@mwild1 mwild1 commented Oct 7, 2021

Part of #60. 100% untested.

@mwild1
Copy link
Member Author

mwild1 commented Nov 12, 2021

How this would be tested:

In snikket.conf, to test a Snikket instance at chat.example.com, but with JIDs @example.com:

SNIKKET_DOMAIN=chat.example.com
SNIKKET_TWEAK_XMPP_DOMAIN=example.com

Prosody will look for certificates for example.com.

@Rijul-A
Copy link
Contributor

Rijul-A commented Nov 19, 2021

Is this ready for testing?

@mwild1
Copy link
Member Author

mwild1 commented Nov 19, 2021

Yes. Not in production though :)

I just went through the Prosody config and changed the domains appropriately. I'm almost 100% certain it won't work - the next step is to identify what's broken so we know what else needs doing.

@mwild1
Copy link
Member Author

mwild1 commented Nov 19, 2021

For example, this PR alone doesn't touch the certificate generation. That's going to be a much harder one to solve (as outlined in #60).

If TWEAK domain is set, use it to create invites and in the cert monitor
to import into prosody.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants