Project

General

Profile

Actions

Feature #4568

closed

Decode v0.3 Data

Added by Junxiao Shi over 6 years ago. Updated about 6 years ago.

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

100%

Estimated time:
3.00 h

Description

In Packet03Transition, Data in v0.3 format makes MetaInfo and Content optional.
This issue is to change ndn::Data class so that the decoder can accept a Data with omitted MetaInfo and Content. Validators shall also accept such packets.
The encoder shall continue to emit empty MetaInfo and Content elements, so that the packets are compatible with both formats.


Related issues 1 (0 open1 closed)

Blocks NFD - Task #4564: Release 0.6.2ClosedAlex Afanasyev05/04/2018

Actions
Actions #1

Updated by Junxiao Shi over 6 years ago

  • Status changed from New to In Progress
  • Assignee set to Junxiao Shi
Actions #2

Updated by Junxiao Shi over 6 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 100
Actions #3

Updated by Junxiao Shi over 6 years ago

  • Status changed from Code review to In Progress
  • % Done changed from 100 to 40

I realize that v0.3 has evolvability requirement, so implementation in patchset1 is incomplete.

Actions #4

Updated by Junxiao Shi over 6 years ago

  • % Done changed from 40 to 100

Change 3668 patchset2 implements TLV evolvability requirement.

Actions #5

Updated by Junxiao Shi over 6 years ago

Actions #6

Updated by Junxiao Shi over 6 years ago

Validators shall also accept a Data with omitted MetaInfo and Content.

I'm confident this would work (just omitting MetaInfo and Content), but I won't add a test case for now.
Test case for this and other possible v0.3 encoding (injecting unrecognized non-critical elements) would come in #4583.

Actions #7

Updated by Davide Pesavento over 6 years ago

  • Status changed from In Progress to Closed
Actions #8

Updated by Davide Pesavento about 6 years ago

  • Tags set to Packet03Transition
Actions

Also available in: Atom PDF