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

doc: epic-sapv #4110

Closed
wants to merge 18 commits into from
Closed

doc: epic-sapv #4110

wants to merge 18 commits into from

Conversation

mawyss
Copy link
Contributor

@mawyss mawyss commented Sep 9, 2021

Specifies EPIC-SAPV (Source Authentication and Path Validation).
EPIC-SAPV enables source authentication at border routers, packet
authentication and path validation at the destination host, and
optional path validation at the source host.


This change is Reviewable

@matzf matzf added the i/proposal A new idea requiring additional input and discussion label Aug 29, 2022
@matzf
Copy link
Contributor

matzf commented Mar 14, 2023

Discussion in the proposal review meeting; as far as we are aware, there is currently no practical requirement to include EPIC-SAPV in this SCION implementation. As this is a major subsystem, adding this would incur both a significant maintenance burden and also increased surface area for bugs and attacks, which we don't think is justified without more concrete evidence for the practicality of this system.
Our recommendation is to build and develop this feature in a fork of scionproto/scion. Perhaps we can find a good place to link to such a fork, so that this can still find sufficient publicity.
Please note that this means that we reject including the implementation of this feature into scionproto/scion for now. But discussions on the design and properties of such a system e.g. in issues in this repository is still welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i/proposal rejected i/proposal A new idea requiring additional input and discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants