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

Clarify wether routers perform checks on segment types (up/down/core) #52

Open
nicorusti opened this issue Aug 25, 2024 · 2 comments · Fixed by #61
Open

Clarify wether routers perform checks on segment types (up/down/core) #52

nicorusti opened this issue Aug 25, 2024 · 2 comments · Fixed by #61
Labels
Prio 3 question Further information is requested
Milestone

Comments

@nicorusti
Copy link
Member

nicorusti commented Aug 25, 2024

In section 1.4 we write:

The type of segment is known to the endpoint but is not visible
is the path header of data packets. Therefore, a SCION router needs
to explicitly verify that these rules were followed correctly.

this sentence actually comes from the book section "5.5 Path Construction (Segment Combinations)".

We then describe checks at routers in 4.2.2. Processing at Routers and there is no mention that routers verify any of the rules in 1.4.

@jiceatscion do you know if routers do any checks at all? If not, we should just remove the second sentence I guess.

@nicorusti nicorusti added question Further information is requested Prio 3 labels Aug 25, 2024
@jiceatscion
Copy link
Contributor

jiceatscion commented Aug 26, 2024 via email

@nicorusti
Copy link
Member Author

I proposed #61 based on your inputs, feel free to review and modify

@nicorusti nicorusti added this to the -03 milestone Oct 8, 2024
@nicorusti nicorusti reopened this Nov 3, 2024
@nicorusti nicorusti modified the milestones: -03, -04 Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prio 3 question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants