Skip to content

fix: enhance TURN server configuration handling in bbb-install.sh#827

Merged
antobinary merged 1 commit intobigbluebutton:v3.0.x-releasefrom
michelleDeko:fix-#825
Apr 2, 2026
Merged

fix: enhance TURN server configuration handling in bbb-install.sh#827
antobinary merged 1 commit intobigbluebutton:v3.0.x-releasefrom
michelleDeko:fix-#825

Conversation

@michelleDeko
Copy link
Copy Markdown
Contributor

fixes #825

The new approach for setting the turnserver is:

  • XML exists and has different host in it: output a warning message but don't change /etc/bigbluebutton/turn-stun-servers.xml
  • XML doesn't exist: create file with the included coturn or the specified external coturn.
  • XML exists but -c was specified: Change file with the specified coturn server and secret.

This prevents the turn-stun-server.xml getting overwritten without specifying "-c" and also fixes the issue where /usr/share/bbb-web/WEB-INF/classes/spring/turn-stun-servers.xml would get overwritten, when bbb-web was updated with apt upgrade, instead of updating BBB via bbb-install

@antobinary
Copy link
Copy Markdown
Member

Thank you @michelleDeko !!

Copy link
Copy Markdown
Member

@antobinary antobinary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BrentBaccala reviewed and posted the result in #825
I am referencing it here so we satisfy the Review step

@antobinary antobinary merged commit f9a054e into bigbluebutton:v3.0.x-release Apr 2, 2026
3 checks passed
@michelleDeko michelleDeko deleted the fix-#825 branch April 6, 2026 17:27
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 this pull request may close these issues.

PR #818 regresses TURN config persistence for local coturn (no -c flag)

2 participants