Project

General

Profile

Actions

Task #2111

closed

ContentType=NACK

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

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

100%

Estimated time:
1.00 h

Description

Define a MetaInfo ContentType for producer generated NACK.

This is an experimental feature that violates NDN-TLV.


Related issues 1 (0 open1 closed)

Related to ndn-cxx - Task #2128: Deduplicate tlv::ContentTypeValue and MetaInfo::TYPE_*ClosedJunxiao Shi

Actions
Actions #1

Updated by Junxiao Shi over 9 years ago

  • Estimated time set to 1.00 h

Although all this takes is to declare a constant in MetaInfo class, I can anticipate lots of objections as similar to #2021, so estimated time is set to 1 hour.

Actions #2

Updated by Anonymous over 9 years ago

My objection to #2021 was partially based on the fact that it's a change to the TLV structure. An application that accidentally pulls one of the experimental Data packets will get a parse error (i.e. exception thrown by the library it uses) if its library isn't aware of the experimental change. This is a serious problem because the app/library will drop the Data packet without telling the application what the "bad" packet was. In other words, the application is in a position where it doesn't know how to avoid the "bad" packet (it was never told its name or implicit digest) and so the only thing it can do is re-request...which will likely just bring the "bad" packet in again. We should probably discuss this problem and potential solutions as a separate topic though (e.g. provide the "bad" packet's digest to the app via a callback).

I don't think there's any such problem here. It sounds like the proposal is to add a new content type value without changing the TLV structure. The only problem would be if parsers enforce acceptable content type values.

Actions #3

Updated by Junxiao Shi over 9 years ago

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

Updated by Junxiao Shi over 9 years ago

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

Updated by Junxiao Shi over 9 years ago

  • Related to Task #2128: Deduplicate tlv::ContentTypeValue and MetaInfo::TYPE_* added
Actions #6

Updated by Junxiao Shi over 9 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF