Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

Turn server documentation - edge case involving Kurento and lt-cred-mech #138

Closed
wants to merge 1 commit into from

Conversation

richp10
Copy link

@richp10 richp10 commented May 5, 2020

I suggest not accepting this PR until a few people have checked and commented. There is considerable confusion and I am not 100% certain I have this right.

@basisbit
Copy link
Contributor

basisbit commented May 5, 2020

If you don't want this PR to be merged yet, please turn it into a draft pull request .
To convert a pull request to a draft, look for the Convert to draft link in the Reviewers section on the pull request’s Conversations tab. Clicking the link displays a modal asking you to confirm the change.

@hex-m
Copy link

hex-m commented Dec 3, 2020

At present, if Kurento needs Turn, it must uses the "lt-cred-mech" authentication mechanism

Our friends from Kurento say:

more advanced credential mechanisms may be used if they are handled by the application code)

@j1elo
Copy link

j1elo commented Dec 3, 2020

At present, if Kurento needs Turn, it must uses the "lt-cred-mech" authentication mechanism

Our friends from Kurento say:

more advanced credential mechanisms may be used if they are handled by the application code)

Note that now I've refreshed the different mechanisms available in Coturn and concluded that I was a bit mistaken by saying "more advanced credential mechanisms", because what I really meant was "non-static password for the long-term credential mechanism":

Kurento/bugtracker#461 (comment)

The other alternative credential mechanism offered by Coturn, the "REST-based authentication secret" (use-auth-secret), has not been tested with Kurento WebRtcEndpoint so I'm not sure if it would actually work the same way it works with the so-called long-term static or dynamic user/passwords mechanism.

@j1elo
Copy link

j1elo commented Apr 16, 2021

Hi! I found this while searching for some other Coturn-related matters, so I thought I'd make a quick comment that might help you resolve this.

This PR's change says:

if Kurento needs Turn, it must uses the "lt-cred-mech" authentication mechanism, which consists of a generated username and password formatted in a very particular way. This requires the lt-cred-mech setting in /etc/turnserver.conf

Kurento does not require the "lt-cred-mech" authentication mechanism; it provides Client API methods to allow the Application dynamically managing the credentials, obtain a TURN user and password, and apply them to each individual WebRtcEndpoint.

All this confusion was caused by very poorly worded docs, in part derived from the people who originally wrote the docs not totally understanding how Coturn works, and me not being up to speed in that matter too after a while later.

Kurento docs now are much clearer in stating that the static config shown is just an example, and in production deployments a dynamic method is probably better for security:

https://doc-kurento.readthedocs.io/en/latest/user/faq.html#how-to-install-coturn

This probably also helps solving any confusion that there might be in bigbluebutton/bbb-install#157

@ffdixon
Copy link
Member

ffdixon commented Jan 15, 2022

Closing this issue as according to the Kurento documentatino they recommend using lt-cred-mech.

@ffdixon ffdixon closed this Jan 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants