Feature #5104
closed
Shorten Nack encoding
Added by Junxiao Shi over 4 years ago. Updated over 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 over 4 years ago
- Related to Feature #4535: Nack format for Interest v0.3 added
Updated by Junxiao Shi over 4 years ago
- Blocked by Feature #4804: Signed Interest v0.3 added
Updated by Junxiao Shi over 4 years ago
- Blocks Feature #5105: PIT lookup with shortened Nack added
Updated by Junxiao Shi over 4 years ago
Why is this blocked by #4804?
This is to prevent merge conflicts.
Updated by Davide Pesavento over 4 years ago
- Blocked by deleted (Feature #4804: Signed Interest v0.3)
Updated by Eric Newberry over 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 over 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 over 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 over 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 over 4 years ago
- Status changed from New to In Progress
Updated by Eric Newberry over 4 years ago
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
Updated by Eric Newberry over 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 over 4 years ago
- Status changed from Code review to Rejected
- Target version deleted (
0.9.0) - % Done changed from 100 to 0