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

Implement localhop forwarding #73

Closed
pulsejet opened this issue Dec 17, 2024 · 2 comments · Fixed by #94
Closed

Implement localhop forwarding #73

pulsejet opened this issue Dec 17, 2024 · 2 comments · Fixed by #94
Labels
fw YaNFD issues

Comments

@pulsejet
Copy link
Collaborator

localhop forwards only one hop (confirm from NFD spec)

@pulsejet pulsejet added bug Something isn't working fw YaNFD issues labels Dec 17, 2024
@pulsejet pulsejet removed the bug Something isn't working label Dec 30, 2024
@yoursunny
Copy link
Member

"/localhop forwards only one hop" is not part of the NDN protocol, but simply a convention adopted by NFD.

In 2016, Prof @beichuan asserted that the forwarder does not need to understand names, but can simply follow routing tables and strategies.
I immediately asked about /localhost and /localhop.
His answer was that, /localhop could be realized by a forwarding strategy that chooses not to forward any Interest received from a non-local face to a non-local face.

Therefore, I'd recommend NDNd forwarder to adopt this approach:

  • Build a forwarding strategy that does not forward an Interest received from a non-local face to a non-local face.
  • Assign this strategy to /localhop in the initial strategy choice.

@pulsejet
Copy link
Collaborator Author

pulsejet commented Jan 3, 2025

@yoursunny I took a go at this but it doesn't cover all use cases. /localhop itself can be best-route (DV) or multicast (NLSR), so we'd need two strategies to support this (too complicated).

Unrelated, accessing the face table creates a import cycle so this will also need a major refactor of the forwarder.

pulsejet added a commit that referenced this issue Jan 3, 2025
pulsejet added a commit that referenced this issue Jan 3, 2025
pulsejet added a commit that referenced this issue Jan 3, 2025
@zjkmxy zjkmxy linked a pull request Jan 3, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fw YaNFD issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants