Re: coturn: specify pidfile and create directory in rc_pre

2023-03-13 Thread Stuart Henderson
On 2023/03/13 10:20, Andre Stoebe wrote: > Hey, > > I just remembered this issue is still open. Sorry for not following up > the last time. > > So here is a diff without all the fancy variables and such. > > I'm running with this change locally since then, but it gets overridden > on upgrade.

Re: coturn: specify pidfile and create directory in rc_pre

2023-03-13 Thread Andre Stoebe
Hey, I just remembered this issue is still open. Sorry for not following up the last time. So here is a diff without all the fancy variables and such. I'm running with this change locally since then, but it gets overridden on upgrade. It would be great if this could make it into the release.

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-06 Thread Antoine Jacoutot
Agreed. — Antoine > On 6 Aug 2022, at 19:07, Stuart Henderson wrote: > > On 2022/08/06 18:00, Omar Polo wrote: >> Andre Stoebe wrote: >>> >>> Stuart Henderson wrote: > On 2022/08/06 11:56, Omar Polo wrote: >> Andre Stoebe wrote: >>> You really don't see the same behaviour, Oma

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-06 Thread Stuart Henderson
On 2022/08/06 18:00, Omar Polo wrote: > Andre Stoebe wrote: > > > > Stuart Henderson wrote: > > > On 2022/08/06 11:56, Omar Polo wrote: > > >> Andre Stoebe wrote: > > >>> You really don't see the same behaviour, Omar? I don't have anything > > >>> set, neither in /etc/rc.conf.local nor in /etc/r

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-06 Thread Omar Polo
Andre Stoebe wrote: > > Stuart Henderson wrote: > > On 2022/08/06 11:56, Omar Polo wrote: > >> Andre Stoebe wrote: > >>> You really don't see the same behaviour, Omar? I don't have anything > >>> set, neither in /etc/rc.conf.local nor in /etc/rc.d/turnserver. > >> > >> Yes, I can see it too now.

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-06 Thread Andre Stoebe
Stuart Henderson wrote: > On 2022/08/06 11:56, Omar Polo wrote: >> Andre Stoebe wrote: >>> You really don't see the same behaviour, Omar? I don't have anything >>> set, neither in /etc/rc.conf.local nor in /etc/rc.d/turnserver. >> >> Yes, I can see it too now. I have a long uptime on that serve

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-06 Thread Stuart Henderson
On 2022/08/06 11:56, Omar Polo wrote: > Andre Stoebe wrote: > > You really don't see the same behaviour, Omar? I don't have anything > > set, neither in /etc/rc.conf.local nor in /etc/rc.d/turnserver. > > Yes, I can see it too now. I have a long uptime on that server and > /tmp/ was very likely

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-06 Thread Omar Polo
Andre Stoebe wrote: > You really don't see the same behaviour, Omar? I don't have anything > set, neither in /etc/rc.conf.local nor in /etc/rc.d/turnserver. Yes, I can see it too now. I have a long uptime on that server and /tmp/ was very likely been cleared at least once. After restarting turn

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-05 Thread Andre Stoebe
Andre Stoebe wrote: > On 05.08.2022 18:28, Omar Polo wrote: >> which version? are you using some custom flags in /etc/rc.conf.local? >> By any chance, did you enabled(/uncommented) the `pidfile' directive >> in /etc/turnserver.conf? > > I'm also running 4.5.2p2 on OpenBSD 7.1-stable without any

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-05 Thread Andre Stoebe
Hey, On 05.08.2022 18:28, Omar Polo wrote: > which version? are you using some custom flags in /etc/rc.conf.local? > By any chance, did you enabled(/uncommented) the `pidfile' directive > in /etc/turnserver.conf? I'm also running 4.5.2p2 on OpenBSD 7.1-stable without any custom flags in /etc/r

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-05 Thread Stuart Henderson
On 2022/08/05 18:28, Omar Polo wrote: > Andre Stoebe wrote: > > Hello, > > > > turnserver tries to create a pidfile at /var/run/turnserver.pid, but > > this fails due to permissions. It falls back to /var/tmp/turnserver.pid, > > so this ends up in /tmp. > > which version? are you using some cus

Re: coturn: specify pidfile and create directory in rc_pre

2022-08-05 Thread Omar Polo
Andre Stoebe wrote: > Hello, > > turnserver tries to create a pidfile at /var/run/turnserver.pid, but > this fails due to permissions. It falls back to /var/tmp/turnserver.pid, > so this ends up in /tmp. which version? are you using some custom flags in /etc/rc.conf.local? By any chance, did yo

coturn: specify pidfile and create directory in rc_pre

2022-08-04 Thread Andre Stoebe
Hello, turnserver tries to create a pidfile at /var/run/turnserver.pid, but this fails due to permissions. It falls back to /var/tmp/turnserver.pid, so this ends up in /tmp. Here's a diff that changes this to /var/run/turnserver/turnserver.pid and creates the directory in rc_pre() before starting