Skip to content
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

Peer user timeout #631

Closed
martinduke opened this issue Aug 7, 2020 · 4 comments
Closed

Peer user timeout #631

martinduke opened this issue Aug 7, 2020 · 4 comments
Labels

Comments

@martinduke
Copy link
Contributor

sec 7.2
perhaps I'm not thinking of this correctly, but should there be a way to query the peer-offered user timeout?

@gorryfair gorryfair added the API label Aug 12, 2020
@mwelzl mwelzl self-assigned this Aug 14, 2020
@tfpauly tfpauly added mappings and removed API labels Aug 14, 2020
@tfpauly
Copy link
Contributor

tfpauly commented Aug 14, 2020

This looks protocol-specific (it's not a generic transport function). QUIC could let you query it in a protocol-specific property.

Proposing moving to the QUIC mapping doc.

mwelzl added a commit that referenced this issue Sep 9, 2020
Closes #625, #627, #629. Partially addresses #631.
@mwelzl
Copy link
Contributor

mwelzl commented Sep 9, 2020

I agree with addressing this in the QUIC mapping doc, but for clarification: this is protocol-specific already (it's about a protocol-specific TCP property), and I added a clarification that the API calls reflect the API specified in Section 3 of RFC 5482 (which doesn't include this functionality).

=> I agree that allowing to query the peer's UTO might be useful, but it's probably not worth adding more text for this, as I also suspect that nobody will implement this for TCP anyway (even the underlying logic from RFC 5482 isn't widely available either AFAIK). If someone does, they are still free to allow querying whatever they like ... but IMO, we shouldn't make this document longer with more protocol-specific elements than we absolutely need to include.

@gorryfair
Copy link
Contributor

Can we close the Peer user timeout with no-action?

@mwelzl mwelzl added discuss and removed discuss labels May 17, 2022
@mwelzl mwelzl closed this as completed May 17, 2022
@mwelzl
Copy link
Contributor

mwelzl commented May 17, 2022

I temporarily gave this a "discuss" label to see if it provokes people to have a 1-minute chat about closing this. This didn't happen, so now I just closed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants