<div dir="auto">What if it were a command line option to delete on start? This way you preserve the existing behaviour while introducing the desired behaviour. It’s up to the user / admin to decide if they want to “force delete” as an example. </div><div dir="auto"><br></div><div dir="auto"><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 25 Apr 2022 at 4:33 pm, <<a href="mailto:shanlei@asiainfo.com">shanlei@asiainfo.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I know the concern, that if we misconfigure the socket file, other file <br>
may be deleted accidentally.<br>
But if the program wont delete the file, bind will fail , that means we <br>
have to delete the file manually before every start. that not a good <br>
option to maintain a system.<br>
<br>
在 2022-04-24 18:05，Александр Поволоцкий wrote：<br>
> Unlink socket that can be in use? I think it's not a good idea.<br>
> <br>
> On 24.04.2022 12:25, <a href="mailto:shanlei@asiainfo.com" target="_blank">shanlei@asiainfo.com</a> wrote:<br>
>> <br>
>> hello,guys,<br>
>> <br>
>> We found if we add a unix socket listener, the file was left in disk <br>
>> when unit got shutdown.<br>
>> If the unit starts again and load configuration from its state folder, <br>
>> it would fail with error msg "file exists",<br>
>> so We think the proper process is to "delete the legacy unix socket <br>
>> before start", and we present following patch:<br>
>> <br>
>> <br>
>> # HG changeset patch<br>
>> # User stdanley <<a href="mailto:shanlei@asiainfo.com" target="_blank">shanlei@asiainfo.com</a>><br>
>> # Date 1650792235 -28800<br>
>> #      Sun Apr 24 17:23:55 2022 +0800<br>
>> # Node ID 29738840c393f12c7d7aa8f3959ed1655b39cd4d<br>
>> # Parent  8e06a879600e83af11345d023c53c2ca445cf82c<br>
>> unlink legacy unix socket file before binding.<br>
>> <br>
>> diff -r 8e06a879600e -r 29738840c393 src/nxt_main_process.c<br>
>> --- a/src/nxt_main_process.c    Tue Apr 12 04:16:00 2022 +0100<br>
>> +++ b/src/nxt_main_process.c    Sun Apr 24 17:23:55 2022 +0800<br>
>> @@ -1142,6 +1142,15 @@<br>
>>      }<br>
>> <br>
>>  #endif<br>
>> +#if (NXT_HAVE_UNIX_DOMAIN)<br>
>> +<br>
>> +    if (sa->u.sockaddr.sa_family == AF_UNIX<br>
>> +        && sa->type == SOCK_STREAM<br>
>> +        && sa->u.sockaddr_un.sun_path[0] != '\0') {<br>
>> +<br>
>> +        (void) unlink(sa->u.sockaddr_un.sun_path);<br>
>> +}<br>
>> +#endif<br>
>> <br>
>>      if (bind(s, &sa->u.sockaddr, sa->socklen) != 0) {<br>
>>          err = nxt_errno;<br>
>> _______________________________________________<br>
>> unit mailing list -- <a href="mailto:unit@nginx.org" target="_blank">unit@nginx.org</a><br>
>> To unsubscribe send an email to <a href="mailto:unit-leave@nginx.org" target="_blank">unit-leave@nginx.org</a><br>
> _______________________________________________<br>
> unit mailing list -- <a href="mailto:unit@nginx.org" target="_blank">unit@nginx.org</a><br>
> To unsubscribe send an email to <a href="mailto:unit-leave@nginx.org" target="_blank">unit-leave@nginx.org</a><br>
_______________________________________________<br>
unit mailing list -- <a href="mailto:unit@nginx.org" target="_blank">unit@nginx.org</a><br>
To unsubscribe send an email to <a href="mailto:unit-leave@nginx.org" target="_blank">unit-leave@nginx.org</a><br>
</blockquote></div></div>