microsoft / microsoft/demikernel
[inetstack] Rely on `SocketAddr` `enum` Rather Than `SocketAddrV4` `struct`
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 1.2k
- Forks
- 151
- PR merge metrics
- No merged PRs in 30d
Description
Description
Catnip uses the Ipv4EndpointSocketAddrV4 structure to refer to remote and local endpoints. While this is fine when supporting IPv4 addresses only, once we add support to IPv6 it is make the stack a mess. Therefore, we should change the stack so as to refer to local and remote endpoints using a more generic type, like IpEndpointSocketAddr.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating Catnip's local and remote endpoint declarations and every use of SocketAddrV4. Trace how those endpoints are passed through the stack, then replace the IPv4-specific type with SocketAddr consistently. Done means the stack uses the generic endpoint type throughout and its existing tests still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- networking
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100