Skip to content

Adds support for UNIX sockets#272

Merged
anarthal merged 22 commits intoboostorg:developfrom
anarthal:feature/246-unix-sockets
Jun 20, 2025
Merged

Adds support for UNIX sockets#272
anarthal merged 22 commits intoboostorg:developfrom
anarthal:feature/246-unix-sockets

Conversation

@anarthal
Copy link
Collaborator

close #246

@anarthal anarthal marked this pull request as ready for review June 15, 2025 11:33
@anarthal anarthal requested a review from mzimbres June 15, 2025 11:33
@mzimbres
Copy link
Collaborator

Many thanks. I will try to finish the review today after kids go to bed :) I think we have to agree on naming, I am more used to the term Unix Domain socket (UDS) than UNIX sockets. I don't know which is better or more popular, I will look at how other libraries name it.

@anarthal
Copy link
Collaborator Author

Many thanks. I will try to finish the review today after kids go to bed :) I think we have to agree on naming, I am more used to the term Unix Domain socket (UDS) than UNIX sockets. I don't know which is better or more popular, I will look at how other libraries name it.

I think the correct term is UNIX domain socket. However, I didn't immediately recognize the UDS acronym as 'UNIX domain socket', so I'd use the full term but avoid 'UDS'.

As a reference:

I can rename config::unix_socket to config::unix_domain_socket or to config::unix_socket_path, and will update all comments to use "UNIX domain sockets"

@anarthal anarthal merged commit 7304d99 into boostorg:develop Jun 20, 2025
19 checks passed
@anarthal anarthal deleted the feature/246-unix-sockets branch June 20, 2025 11:23
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.

unix sockets support?

2 participants