Feature #5104
closed
Shorten Nack encoding
Added by Junxiao Shi about 4 years ago. Updated almost 4 years ago.
0%
Description
When encoding Nack, only include Name, CanBePrefix, MustBeFresh, ForwardingHint, and Nonce elements of the Interest. Omit other fields.
Change Interest-to-Nack matching functions accordingly.
Updated by Junxiao Shi about 4 years ago
- Related to Feature #4535: Nack format for Interest v0.3 added
Updated by Junxiao Shi about 4 years ago
- Blocked by Feature #4804: Signed Interest v0.3 added
Updated by Junxiao Shi about 4 years ago
- Blocks Feature #5105: PIT lookup with shortened Nack added
Updated by Junxiao Shi about 4 years ago
Why is this blocked by #4804?
This is to prevent merge conflicts.
Updated by Davide Pesavento about 4 years ago
- Blocked by deleted (Feature #4804: Signed Interest v0.3)
Updated by Eric Newberry about 4 years ago
It seems this will require changes to both ndn-cxx and NFD, since they each have their own Nack encoding mechanisms in their respective face systems. Is this correct?
Updated by Junxiao Shi about 4 years ago
Eric Newberry wrote:
It seems this will require changes to both ndn-cxx and NFD, since they each have their own Nack encoding mechanisms in their respective face systems. Is this correct?
No. NFD change is #5105.
Updated by Eric Newberry about 4 years ago
So, if I'm understanding this issue correctly, the Interest-to-Nack matching function will need to be changed so that all five of the fields match, instead of just the first three as in the Interest-to-Interest matching function?
Updated by Junxiao Shi about 4 years ago
So, if I'm understanding this issue correctly, the Interest-to-Nack matching function will need to be changed so that all five of the fields match, instead of just the first three as in the Interest-to-Interest matching function?
I'm uncertain about library behavior. You can check how Python library is doing things.
Forwarder behavior is answered in #5105-3.
Updated by Eric Newberry about 4 years ago
- Status changed from New to In Progress
Updated by Eric Newberry about 4 years ago
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
Updated by Eric Newberry almost 4 years ago
At the NDN Platform Call on May 22, 2020, it was decided to revert this design decision and instead encode the entire original Interest in Nacks. Therefore, this change is not needed.
Updated by Davide Pesavento almost 4 years ago
- Status changed from Code review to Rejected
- Target version deleted (
0.9.0) - % Done changed from 100 to 0
Updated by Davide Pesavento almost 4 years ago
- Assignee deleted (
Eric Newberry)