Alejandro Colomar

For patch 1: `Reviewed-by: Alejandro Colomar `

@remicollet Please remove the preprocessor conditionals (since it works with old OpenSSL versions too) and the pragma (since we don't get the warning, as Andrew reported). Also, please sign the...

Merged. Thanks!

Hi! I have a branch where I'm experimenting with adding it (BTW, the flag was added to GCC in version 10). The problem is that there are so many warnings,...

> In my feelings, in the function `nxt_socket_bind()`, there are a few places that are worth reworking. > > ``` > #if NXT_HAVE_UNIX_DOMAIN > if (sa->u.sockaddr.sa_family == AF_UNIX) { >...

> > 4. This will make `nxt_socket_bind()` dependent on `nxt_fs_mkdir_all()`. Or say that making `socket` dependent on `file`. It seems it's unreasonable. And users will be able to create directories...

What bind does is bind a socket file descriptor to a filesystem path. That filesystem path includes a file and its parent directory. I don't think creating a file is...

> Just to wonder if DevOps can do the thing of creating directories by themself? Do you mean manually create the directory? It's not reasonable, because at reboot, the OS...

> > What bind does is bind a socket file descriptor to a filesystem path. That filesystem path includes a file and its parent directory. I don't think creating a...