add methods to TCP and UDP sockets to modify hop limit by kckeiks · Pull Request #94678 · rust-lang/rust

kckeiks · 2022-03-06T18:06:15Z

adds methods to set value of IPV6_UNICAST_HOPS and IPV6_MULTICAST_HOPS on ipv6 sockets
I added some noop methods for different systems to keep things consistent. Maybe someone could please clarify if these noop funcs are needed? and, if so, why? Just for my own learning.

This is my first addition of a new feature so let me know if I missed something in the process. I read the rustc dev guide about implementing new features but since the change here is simple, it suggested that a PR would be enough.

Fixes #47727

rust-highfive · 2022-03-06T18:06:18Z

r? @kennytm

(rust-highfive has picked a reviewer for you, use r? to override)

kennytm · 2022-03-07T06:42:05Z

library/std/src/net/tcp.rs

+    /// stream.set_multicast_hlim_v6(88).expect("set_hop_limit_v6 call failed");
+    /// ```
+    #[unstable(feature = "ipv6_hop_limit", issue = "47727")]
+    pub fn set_multicast_hlim_v6(&self, limit: u32) -> io::Result<()> {


why thi is "hlim" here but "hop_limit" for unicast

I was trying to keep the length of the method name from being too long. Should I change it to "hop_limit"?

Yes. Rust doesn't generally abbreviate.

I updated the method names.

the8472 · 2022-05-21T10:24:39Z

IPV6_UNICAST_HOPS and IPV6_MULTICAST_HOPS are standardized in IETF RFC 3493 so this should be a good cross-platform addition.

Since it adds new API surface: r? rust-lang/libs-api

bors · 2022-06-07T08:05:05Z

☔ The latest upstream changes (presumably #95565) made this pull request unmergeable. Please resolve the merge conflicts.

joshtriplett · 2022-10-03T13:21:11Z

This seems fine, assuming it passes CI. Let's see if the requisite constants are defined on all platforms...

@bors r+

rust-log-analyzer · 2022-10-04T03:29:44Z

The job dist-various-2 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)


error: unused variable: `limit`
   --> library/std/src/sys/sgx/net.rs:456:36
    |
456 |     pub fn set_hop_limit_v6(&self, limit: u32) -> io::Result<()> {
    |                                    ^^^^^ help: if this is intentional, prefix it with an underscore: `_limit`
    |
    = note: `-D unused-variables` implied by `-D warnings`
error: unused variable: `limit`
   --> library/std/src/sys/sgx/net.rs:464:46
    |
    |
464 |     pub fn set_multicast_hop_limit_v6(&self, limit: u32) -> io::Result<()> {
    |                                              ^^^^^ help: if this is intentional, prefix it with an underscore: `_limit`
[RUSTC-TIMING] std test:false 3.547
warning: `std` (lib) generated 1 warning
error: could not compile `std` due to 2 previous errors; 1 warning emitted
Build completed unsuccessfully in 0:22:44

emilyalbini · 2022-10-05T09:13:37Z

@bors r-

Queue got messed up after a synchronize.

bors · 2022-10-15T05:02:47Z

☔ The latest upstream changes (presumably #98033) made this pull request unmergeable. Please resolve the merge conflicts.

mominul · 2022-10-15T07:59:09Z

library/std/src/net/tcp.rs

+    /// stream.set_hop_limit_v6(88).expect("set_hop_limit_v6 call failed");
+    /// ```
+    #[unstable(feature = "ipv6_hop_limit", issue = "47727")]
+    pub fn set_hop_limit_v6(&self, limit: u32) -> io::Result<()> {


Hop Limit is an 8bit header field. So using u8 instead of u32 would reflect that better and avoid API misusage like #73158.

Dylan-DPC · 2022-11-12T09:14:04Z

@kckeiks any updates on this?

kckeiks · 2022-11-12T17:43:25Z

@kckeiks any updates on this?

Sorry, I've been super busy, I will push an update sometime this week though.

kckeiks · 2023-01-20T04:06:57Z

Just following up. My apologies, work has been keeping me exceedingly busy so I'm not sure when I'll have some bandwidth to resolve this. :/

Dylan-DPC · 2023-01-20T10:07:01Z

Just following up. My apologies, work has been keeping me exceedingly busy so I'm not sure when I'll have some bandwidth to resolve this. :/

That's fine, can understand. I am going to close this as it is inactive. If you find the time feel free to reöpen it or preferably create a new PR and we can take it from there. Thanks

…=dtolnay Add methods to TCP and UDP sockets to modify hop limit (refresh of rust-lang#94678) rust-lang#94678 was closed because of time constraints of the original author (`@kckeiks).` This PR is a refresh of rust-lang#94678 where the original proposed changes are ported to the current status of the standard library. In addition, this PR: - implements the change suggested in rust-lang#94678 (comment) - defines `set_multicast_hop_limit_v6` and `multicast_hop_limit_v6 ` only for UDP sockets and not for TCP sockets as originally proposed Issue: rust-lang#139166

…=dtolnay Add methods to TCP and UDP sockets to modify hop limit (refresh of rust-lang#94678) rust-lang#94678 was closed because of time constraints of the original author (``@kckeiks).`` This PR is a refresh of rust-lang#94678 where the original proposed changes are ported to the current status of the standard library. In addition, this PR: - implements the change suggested in rust-lang#94678 (comment) - defines `set_multicast_hop_limit_v6` and `multicast_hop_limit_v6 ` only for UDP sockets and not for TCP sockets as originally proposed Issue: rust-lang#139166

…enton Rollup of 12 pull requests Successful merges: - rust-lang#138744 (Add methods to TCP and UDP sockets to modify hop limit (refresh of rust-lang#94678)) - rust-lang#138962 (Expect an array when expected and acutal types are both arrays during cast) - rust-lang#139001 (add `naked_functions_rustic_abi` feature gate) - rust-lang#139379 (Use delayed bug for normalization errors in drop elaboration) - rust-lang#139582 (Various coercion cleanups) - rust-lang#139628 (Suggest remove redundant `$()?` around `vis`) - rust-lang#139644 (Micro-optimize `InstSimplify`'s `simplify_primitive_clone`) - rust-lang#139666 (cleanup `mir_borrowck`) - rust-lang#139674 (In `rustc_mir_transform`, iterate over index newtypes instead of ints) - rust-lang#139695 (compiletest: consistently use `camino::{Utf8Path,Utf8PathBuf}` throughout) - rust-lang#139722 (Move some things to rustc_type_ir) - rust-lang#139740 (Convert `tests/ui/lint/dead-code/self-assign.rs` to a known-bug test) r? `@ghost` `@rustbot` modify labels: rollup

…=dtolnay Add methods to TCP and UDP sockets to modify hop limit (refresh of rust-lang#94678) rust-lang#94678 was closed because of time constraints of the original author (`@kckeiks).` This PR is a refresh of rust-lang#94678 where the original proposed changes are ported to the current status of the standard library. In addition, this PR: - implements the change suggested in rust-lang#94678 (comment) - defines `set_multicast_hop_limit_v6` and `multicast_hop_limit_v6 ` only for UDP sockets and not for TCP sockets as originally proposed Issue: rust-lang#139166

Rollup of 9 pull requests Successful merges: - #134696 (Implement `normalize_lexically`) - #138744 (Add methods to TCP and UDP sockets to modify hop limit (refresh of #94678)) - #140539 (Simplify `attribute_groups`) - #140863 ([rustdoc] Unify type aliases rendering with other ADT) - #140936 (Clarify WTF-8 safety docs) - #140952 (Specify that split_ascii_whitespace uses the same definition as is_ascii_whitespace) - #141472 (Attempt to improve the `std::fs::create_dir_all` docs related to atomicity) - #141502 (ci: move PR job x86_64-gnu-tools to codebuild) - #141559 (const-check: stop recommending the use of rustc_allow_const_fn_unstable) r? `@ghost` `@rustbot` modify labels: rollup

Add methods to TCP and UDP sockets to modify hop limit (refresh of #94678)

kckeiks added 4 commits March 6, 2022 12:58

add setters and getters for ipv6 socket hop limit

ac288b4

add setters for hop limit to tcp listener and stream and update tests

44c2226

add noop methods for other sys

c60ab7b

add unicast and multicast hop limit option for windows

090c1c9

rust-highfive assigned kennytm Mar 6, 2022

rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 6, 2022

This comment has been minimized.

Sign in to view

cleanup - remove whitespace

6d0d29e

This comment has been minimized.

Sign in to view

fix broken intra doc links

36096cf

kckeiks changed the title ~~add methods to TCP and UDP sockets modify hop limit~~ add methods to TCP and UDP sockets to modify hop limit Mar 6, 2022

kennytm reviewed Mar 7, 2022

View reviewed changes

rename multicast methods

b3e9e94

kckeiks requested review from joshtriplett and kennytm March 15, 2022 21:08

JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 24, 2022

apiraino added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label May 19, 2022

the8472 added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. and removed T-libs Relevant to the library team, which will review and decide on the PR/issue. labels May 21, 2022

rust-highfive assigned joshtriplett and unassigned kennytm May 21, 2022

JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 3, 2022

JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 13, 2022

bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 4, 2022

bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 5, 2022

mominul reviewed Oct 15, 2022

View reviewed changes

Dylan-DPC closed this Jan 20, 2023

Dylan-DPC added S-inactive Status: Inactive and waiting on the author. This is often applied to closed PRs. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 20, 2023

Mallets added a commit to Mallets/rust that referenced this pull request Mar 20, 2025

Refresh of rust-lang#94678

858af9e

Mallets mentioned this pull request Mar 20, 2025

Add methods to TCP and UDP sockets to modify hop limit (refresh of #94678) #138744

Open

Mallets mentioned this pull request Mar 31, 2025

Tracking Issue for #138744: Add methods to TCP and UDP sockets to modify IPv6 hop limits #139166

Open

4 tasks

Mallets added a commit to Mallets/rust that referenced this pull request Mar 31, 2025

Refresh of rust-lang#94678

27b67d4

dtolnay pushed a commit to Mallets/rust that referenced this pull request Apr 13, 2025

Refresh of rust-lang#94678

5ce4d0d

ChrisDenton mentioned this pull request Apr 13, 2025

Rollup of 13 pull requests #139754

Closed

ChrisDenton mentioned this pull request Apr 13, 2025

Rollup of 12 pull requests #139755

Closed

jhpratt mentioned this pull request May 25, 2025

Rollup of 9 pull requests #141561

Closed

rust-bors bot added a commit that referenced this pull request Sep 4, 2025

Auto merge of #138744 - Mallets:set_ipv6_sock_hop_limit, r=<try>

6b5fa05

Add methods to TCP and UDP sockets to modify hop limit (refresh of #94678)

Mallets added a commit to Mallets/rust that referenced this pull request Sep 20, 2025

Refresh of rust-lang#94678

38fc880

Uh oh!

Conversation

kckeiks commented Mar 6, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rust-highfive commented Mar 6, 2022

Uh oh!

This comment has been minimized.

This comment has been minimized.

kennytm Mar 7, 2022

Choose a reason for hiding this comment

Uh oh!

kckeiks Mar 7, 2022

Choose a reason for hiding this comment

Uh oh!

joshtriplett Mar 8, 2022

Choose a reason for hiding this comment

Uh oh!

kckeiks Mar 8, 2022

Choose a reason for hiding this comment

Uh oh!

the8472 commented May 21, 2022

Uh oh!

bors commented Jun 7, 2022

Uh oh!

joshtriplett commented Oct 3, 2022

Uh oh!

rust-log-analyzer commented Oct 4, 2022

Uh oh!

emilyalbini commented Oct 5, 2022

Uh oh!

bors commented Oct 15, 2022

Uh oh!

mominul Oct 15, 2022

Choose a reason for hiding this comment

Uh oh!

Dylan-DPC commented Nov 12, 2022

Uh oh!

kckeiks commented Nov 12, 2022

Uh oh!

kckeiks commented Jan 20, 2023

Uh oh!

Dylan-DPC commented Jan 20, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

12 participants

kckeiks commented Mar 6, 2022 •

edited

Loading