Project

General

Profile

Actions

Feature #5187

closed

Update ForwardingHint to 1*Name format

Added by Junxiao Shi over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Protocol
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
18.00 h

Description

Previous format (published in 2017):

ForwardingHint = FORWARDING-HINT-TYPE TLV-LENGTH 1*Delegation
Delegation = DELEGATION-TYPE TLV-LENGTH Preference Name

Change to new format (published in 2021):

ForwardingHint = FORWARDING-HINT-TYPE TLV-LENGTH 1*Name

When feasible, accept previous format for decoding, in order to avoid a flag day on network upgrades.


Related issues 2 (0 open2 closed)

Blocked by NDN Specifications - Task #5178: Remove Preference field from DelegationClosedAlex Afanasyev

Actions
Blocks NFD - Task #5172: Release NFD 22.02 / ndn-cxx 0.8.0ClosedAlex Afanasyev

Actions
Actions #1

Updated by Junxiao Shi over 2 years ago

  • Blocked by Task #5178: Remove Preference field from Delegation added
Actions #3

Updated by Junxiao Shi over 2 years ago

ndn-tools:
https://gerrit.named-data.net/c/ndn-tools/+/6565

NFD: will come in next few days.

ndns: will come in next few days.

python-ndn: I don't understand the encoding module. The maintainer has to do it.

Actions #4

Updated by Junxiao Shi over 2 years ago

NFD:
https://gerrit.named-data.net/c/NFD/+/6567

ndns:
https://gerrit.named-data.net/c/ndns/+/6568
All tests pass, but I don't understand the code so I don't know whether it's correct.
I'm seeing many problems in this codebase but I will not improve further.

Actions #5

Updated by Junxiao Shi over 2 years ago

  • % Done changed from 40 to 80
Actions #7

Updated by Junxiao Shi over 2 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 80 to 100
Actions #8

Updated by Davide Pesavento over 2 years ago

  • Status changed from Code review to Closed
Actions #9

Updated by Davide Pesavento over 2 years ago

  • Blocks Task #5172: Release NFD 22.02 / ndn-cxx 0.8.0 added
Actions

Also available in: Atom PDF