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

PATH_STANDBY PATH_AVAILABLE Sequence number spaces #312

Open
gloinul opened this issue Mar 16, 2024 · 2 comments
Open

PATH_STANDBY PATH_AVAILABLE Sequence number spaces #312

gloinul opened this issue Mar 16, 2024 · 2 comments

Comments

@gloinul
Copy link

gloinul commented Mar 16, 2024

Section 8.3:
A variable-length integer specifying the sequence number assigned for this PATH_STANDBY frame. The sequence number space is shared with the PATH_AVAILABLE frame and the sequence number MUST be monotonically increasing generated by the sender of the PATH_STANDBY frame in the same connection. The receiver of the PATH_STANDBY frame needs to use and compare the sequence numbers separately for each Destination Connection ID Sequence Number.

So are sequnce number spaces per CID? But shared across the PATH_STANDBY and PATH_AVAILABLE

@mirjak
Copy link
Collaborator

mirjak commented Mar 18, 2024

No one number space but you only compare to the number that you previously received for the same CID.

E.g. you received

SEQ=1 CID=1
SEQ=2 CID=2
SEQ=4 CID=1
SEQ=3 CID=2

That means you have to update both paths and not throw away the frame with SEQ=3

@mirjak
Copy link
Collaborator

mirjak commented May 14, 2024

I think this issue is still true but you use now one sequence number space for all path IDs (instead of CIDs). I guess we could also define a separate sequence number path for each path ID now that the path IDs are stable, however, I don't think that makes a real difference.

However, I guess we should clarify this in the draft?

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

No branches or pull requests

2 participants