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

Cannot configure path for coturn certificates if HTTPS Proxy is disabled #137

Closed
Zocker1999NET opened this issue Jul 26, 2021 · 1 comment · Fixed by #144
Closed

Cannot configure path for coturn certificates if HTTPS Proxy is disabled #137

Zocker1999NET opened this issue Jul 26, 2021 · 1 comment · Fixed by #144

Comments

@Zocker1999NET
Copy link

docker/scripts/setup

Lines 139 to 140 in 0dd3d9a

sed -i "s/#COTURN_TLS_CERT_PATH=.*/COTURN_TLS_CERT_PATH=$CERTPATH/" .env
sed -i "s/#COTURN_TLS_KEY_PATH=.*/COTURN_TLS_KEY_PATH=$KEYPATH/" .env

The setup script does not allow users to set a correct path to the certificates for coturn as sed will interpret any slash / as end of a replacement. (my env is German, that's because sed's error is also in German):

Please enter path to cert.pem: /tmp/cert.pem                                                                                                                                                                                                                                                                                  
Please enter path to key.pem: /tmp/key.pem                                                                                                                                                                                                                                                                                    
Should a Prometheus exporter be included? (y/n): n                                                                                                                                                                                                                                                                            
Please enter the domain name: bbb.example.com                                                                                                                                                                                                                                                                              
Should the recording feature be included?                                                                                                                                                                                                                                                                                     
   IMPORTANT: this is currently a big privacy issues, because it will                                                                                                                                                                                                                                                         
   record everything which happens in the conference, even when the button                                                                                                                                                                                                                                                    
   suggets, that it does not.                                                                                                                                  
   make sure that you awalys get peoples consent, before they join a room!
   https://github.com/bigbluebutton/bigbluebutton/issues/9202                                                                                                  
Choice (y/n): n                                              
Is 109.193.183.143 your external IPv4 address? (y/n): y                                                                                                        
Is 2a02:8071:319b:b200:a696:4786:be72:7eed your external IPv6 address? (y/n): y
sed: -e Ausdruck #1, Zeichen 50: Unbekannte Option für »s«

sed's Error roughly translated: sed: -e expression #1, char 50, unknown option for »s«

@mghadam
Copy link
Contributor

mghadam commented Aug 2, 2021

#144

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

Successfully merging a pull request may close this issue.

2 participants