Galène videoconferencing server discussion list archives
 help / color / mirror / Atom feed
From: "Toke Høiland-Jørgensen" <toke@toke.dk>
To: Juliusz Chroboczek <jch@irif.fr>
Cc: galene@lists.galene.org
Subject: [Galene] Re: Heads-up: built-in TURN server
Date: Tue, 19 Jan 2021 00:05:43 +0100	[thread overview]
Message-ID: <87lfcprgl4.fsf@toke.dk> (raw)
In-Reply-To: <87eeihyipm.wl-jch@irif.fr>

Juliusz Chroboczek <jch@irif.fr> writes:

>> What about when the external TURN server is on a different IP, so Galene
>> has no problem starting up the internal one, but it's firewalled off to
>> clients can't connect to it? In that case presumably it'll just be
>> another TURN candidate offered to clients which will fail?
>
> The external TURN server will be tried first, so the latency of successful
> connections will not increase.  You'll just take more time to fail.

Right, OK.

>> (I think Firefox emits a warning recommending a max of two candidates)?
>
> Yeah, I'm not sure why.  I see no reason for that in the ICE spec, but
> perhaps Firefox are doing proprietary magic.
>
> At any rate, I'd expect somebody who goes through the trouble of setting
> up an instance of coturn to be willing to add a command-line option to
> Galène.  I'd like to understand why you and Michael appear to disagree,
> perhaps I'm missing something obvious.

Sure, I can add it, of course :)

It's just a bit of extra work; specifically, I have to go in and update
the systemd unit file on update, and if I wasn't paying attention to the
mailing list I would likely have missed this addition and end up running
with a "broken" config.

And also, from a "least surprise" PoV I think it's a bit odd to
second-guess the user: if I configured a list of TURN servers, I would
expect that that is the list that would be used, and not have it amended
without an explicit opt-in.

Just to be clear, I think turning on the TURN server by default is
totally fine to make it easier to deploy Galene! It's only the "override
user config" aspect I'm objecting to (and it's not a terribly strong
objection, at that).

-Toke

  reply	other threads:[~2021-01-18 23:05 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-18 21:26 [Galene] " Juliusz Chroboczek
2021-01-18 21:38 ` [Galene] " Toke Høiland-Jørgensen
2021-01-18 21:44   ` Juliusz Chroboczek
2021-01-18 21:54     ` Toke Høiland-Jørgensen
2021-01-18 22:03       ` Michael Ströder
2021-01-18 22:04       ` Juliusz Chroboczek
2021-01-18 22:27         ` Toke Høiland-Jørgensen
2021-01-18 22:37           ` Juliusz Chroboczek
2021-01-18 23:05             ` Toke Høiland-Jørgensen [this message]
2021-01-18 23:39               ` Michael Ströder
2021-01-19  0:22               ` Juliusz Chroboczek
2021-01-19 11:47                 ` Toke Høiland-Jørgensen
2021-01-19 11:52                   ` Juliusz Chroboczek
2021-01-19 12:12                     ` Toke Høiland-Jørgensen
2021-01-19 12:37                       ` Michael Ströder
2021-01-19 12:48                       ` Juliusz Chroboczek
2021-01-19 13:07                         ` Toke Høiland-Jørgensen
2021-01-19  8:33 ` Rémi Nollet
2021-01-19 11:00   ` Gabriel Kerneis
2021-01-19 11:21   ` Juliusz Chroboczek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://lists.galene.org/postorius/lists/galene.lists.galene.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87lfcprgl4.fsf@toke.dk \
    --to=toke@toke.dk \
    --cc=galene@lists.galene.org \
    --cc=jch@irif.fr \
    --subject='[Galene] Re: Heads-up: built-in TURN server' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox