Project

General

Profile

Task #4586

Signature: redefine signed portion considering unrecognized non-critical TLV

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

Status:
Closed
Priority:
Normal
Start date:
Due date:
% Done:

100%

Estimated time:

Description

Packet Format v0.3 defines SignatureValue to be a signature over Name+MetaInfo+Content+SignatureInfo. For example, with SignatureType=3, it is ECDSA over SHA256{Name, MetaInfo, Content, SignatureInfo}. This definition makes it impossible to introduce new elements covered by signature in a compatible way.

To restore the design intention of SignatureValue, the definition shall be updated as follows:

  • All elements before SignantureValue are covered by signature.
  • All elements after SignatureValue are not covered by signature.

This enables future protocol versions to introduce new signed elements by adding them before SignatureValue or introduce new unsigned elements by adding them after SignatureValue.


Related issues

Related to NDN Specifications - Task #4587: Interest+Data: forbid unrecognized non-critical TLV before/inside NameClosedJunxiao Shi04/18/2018

Actions
Blocks ndn-cxx - Bug #4583: verification-helpers: cannot handle Data with unrecognized non-critical TLV elementsClosedEric Newberry

Actions
#1

Updated by Junxiao Shi over 2 years ago

  • Blocks Bug #4583: verification-helpers: cannot handle Data with unrecognized non-critical TLV elements added
#2

Updated by Junxiao Shi over 2 years ago

  • Related to Task #4587: Interest+Data: forbid unrecognized non-critical TLV before/inside Name added
#3

Updated by Davide Pesavento over 1 year ago

  • Start date deleted (04/18/2018)
#4

Updated by Junxiao Shi 10 months ago

  • Status changed from New to In Progress
  • Assignee set to Junxiao Shi
  • % Done changed from 0 to 100
#5

Updated by Junxiao Shi 8 months ago

  • Assignee deleted (Junxiao Shi)

I will no longer work on this issue due to disagreements with code reviewers.
Others are welcome to take over.

#6

Updated by Davide Pesavento 4 months ago

  • Status changed from In Progress to Code review
  • Assignee set to Davide Pesavento
#7

Updated by Davide Pesavento 4 months ago

  • Status changed from Code review to Closed

Also available in: Atom PDF