-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow creating sockets from raw file descriptors #6894
Conversation
Use case is valid, but CI is red. |
Needs fix and maybe a spec for the described behaviour. |
@myfreeweb Are you still working on this? |
430e760
to
ccf28cb
Compare
I don't understand why that test is failing.. |
@myfreeweb The added I'd suggest to make This should be changed uniquely in all fd constructors. |
It's a shame that such a change breaks only one mostly unrelated SSL spec 😢 |
1dd3a5c
to
061f5f9
Compare
Passes now |
Awesome 👍 Could you add a doc to the now public |
This is useful for socket activation, sandboxes, etc.
061f5f9
to
953e106
Compare
done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This is useful for socket activation, sandboxes/fd-passing, etc.