search

processone / docker-ejabberd

Set of ejabberd Docker images
95 stars 77 forks source link

STUN/TURN & SIP #38

Closed madmath03 closed 5 years ago

madmath03 commented 5 years ago

:wrench: Enable STUN/TURN and SIP by default

I would understand if you do not want to add all modules in the docker container, but I was forced to fork the repo just to enable this so I feel like I should at least share the idea 😉

mremond commented 5 years ago

I think this is a good idea to enabled those module in the container. If you want to submit a PR, we will consider it :) Thanks !

madmath03 commented 5 years ago

Great 😃 I'm still working on this because, even though I added stun and sip in vars.config, I still get an error message :

[critical] Listening module ejabberd_stun is not available: ejabberd is not compiled with STUN/TURN support
[critical] Listening module ejabberd_sip is not available: ejabberd is not compiled with SIP support

I guess I'm missing something but don't know what... any idea ?

EDIT: Got it, just some silly errors on my side.

I should be able provide this PR with some sample STUN + SIP config soon.

madmath03 commented 5 years ago

@mremond STUN/TURN and SIP should be ready with this.

I also added the mod_sip to the modules and a sample config for the listeners (did not had the time to test it but it matches the official doc).

mremond commented 5 years ago

Thanks, we will have a look 👍

cromain commented 5 years ago

This looks good for merge. Thanks.