Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Regarding allowing guest users to access the TURN server login info#1968

Closed
magiccpp1 wants to merge 3 commits into
matrix-org:developfrom
magiccpp1:develop
Closed

Regarding allowing guest users to access the TURN server login info#1968
magiccpp1 wants to merge 3 commits into
matrix-org:developfrom
magiccpp1:develop

Conversation

@magiccpp1
Copy link
Copy Markdown

Hi ,

we would like to allow the guest users to access the TURN server login info, please review the minor change on voip.py. thanks.

magiccpp1 added 3 commits March 6, 2017 09:44
…are allowed to fetch turn server's info

Signed-off-by: Ken Dai <ken.dai@ericsson.com>
…t users are allowed to fetch turn server's info"

This reverts commit 870307d.
Sign-off-by: Ken Dai <ken.dai@ericsson.com>
@matrixbot
Copy link
Copy Markdown
Member

Can one of the admins verify this patch?

2 similar comments
@matrixbot
Copy link
Copy Markdown
Member

Can one of the admins verify this patch?

@matrixbot
Copy link
Copy Markdown
Member

Can one of the admins verify this patch?

@richvdh
Copy link
Copy Markdown
Member

richvdh commented Mar 15, 2017

@matrixbot: ok to test

@ara4n
Copy link
Copy Markdown
Member

ara4n commented Mar 15, 2017

[cue huge debate over security implications of letting guests relay traffic via TURN if they haven't completed a captcha]

@ara4n
Copy link
Copy Markdown
Member

ara4n commented Mar 15, 2017

The conclusions here were:

I've documented the security implications and made this a configurable setting, and PR'd it as #2011, which supersedes this one. Thanks for bringing this into the spotlight! :)

@ara4n ara4n closed this Mar 15, 2017
@Rugvip
Copy link
Copy Markdown

Rugvip commented Mar 15, 2017

Ok, thanks :) And just to fix some confusion, the PR is #2011, not #2010

@ara4n
Copy link
Copy Markdown
Member

ara4n commented Mar 15, 2017

oops, thanks; fixed

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.

5 participants