Skip to content

Add support for getting SO_COOKIE socket option on linux#437

Merged
Thomasdezeeuw merged 1 commit intorust-lang:masterfrom
bleggett:bleggett/add-socket-cookie
Apr 26, 2023
Merged

Add support for getting SO_COOKIE socket option on linux#437
Thomasdezeeuw merged 1 commit intorust-lang:masterfrom
bleggett:bleggett/add-socket-cookie

Conversation

@bleggett
Copy link
Contributor

@bleggett bleggett commented Apr 25, 2023

Add support for getting the socket cookie on Linux via the SO_COOKIE sockopt (#436)

There is only a getter, as the cookie is managed/created by the kernel itself.

@bleggett bleggett force-pushed the bleggett/add-socket-cookie branch 3 times, most recently from e5d6564 to 675b2ac Compare April 25, 2023 17:00
Copy link
Collaborator

@Thomasdezeeuw Thomasdezeeuw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a quick smoke test to tests/socket.rs?

@bleggett bleggett force-pushed the bleggett/add-socket-cookie branch from 675b2ac to 074cfbe Compare April 26, 2023 18:41
@bleggett
Copy link
Contributor Author

@Thomasdezeeuw done, ty!

Signed-off-by: Benjamin Leggett <benjamin.leggett@solo.io>
@bleggett bleggett force-pushed the bleggett/add-socket-cookie branch from 074cfbe to 757b96b Compare April 26, 2023 20:01
@Thomasdezeeuw Thomasdezeeuw merged commit 488441a into rust-lang:master Apr 26, 2023
@Thomasdezeeuw
Copy link
Collaborator

Thanks @bleggett

@bleggett bleggett deleted the bleggett/add-socket-cookie branch April 26, 2023 20:10
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.

2 participants