Project

General

Profile

Actions

Feature #3197

open

Wireshark dissector: NDNLPv2

Added by Junxiao Shi over 8 years ago. Updated over 7 years ago.

Status:
Feedback
Priority:
Normal
Assignee:
-
Start date:
Due date:
% Done:

80%

Estimated time:
3.00 h

Description

Dissect NDNLPv2 LpPacket.

This issue includes:

  • display the structure of LpPacket
  • recognize these fields and their values: Fragment, Sequence, FragIndex, FragCount, Nack, NackReason, NextHopFaceId, CachePolicy, CachePolicyType, IncomingFaceId
  • for an unknown field that can be ignored, display "unknown field (ignored)"
  • for an unknown field that would cause packet to be dropped, display "unknown field " in red
  • for a field that appears out-of-order, display "out-of-order field " in red
  • invoke NDN network layer packet dissector on Fragment body, without attempting to reassemble fragments

Related issues 1 (1 open0 closed)

Blocks ndn-tools - Feature #3198: Wireshark dissector: NDNLPv2 reassemblyNew

Actions
Actions #1

Updated by Junxiao Shi over 8 years ago

  • Description updated (diff)
Actions #2

Updated by Junxiao Shi over 8 years ago

  • Blocks Feature #3198: Wireshark dissector: NDNLPv2 reassembly added
Actions #3

Updated by Alex Afanasyev over 7 years ago

  • Status changed from New to In Progress
  • Assignee set to zipeng wang
  • % Done changed from 0 to 40
Actions #4

Updated by Alex Afanasyev over 7 years ago

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

Partial implementation of this task here: https://gerrit.named-data.net/#/c/3254/

Actions #5

Updated by Junxiao Shi over 7 years ago

  • Status changed from Code review to In Progress

If the implementation is "partial":

  • Issue status should remain as InProgres. CodeReview should be used only if the final commit has been uploaded.
  • Indicate exactly which features have been implemented (refer to the list of requirements in spec). This allows a reviewer to look at these features only, and allows a future developer to make another commit for remaining features.
Actions #6

Updated by Alex Afanasyev over 7 years ago

  • Status changed from In Progress to Code review

Issue is not in progress, but in code review! Please don't make up random rules.

The partial implementation includes:

  • Displaying the structure of LpPacket
  • Recognizing fields and their values: Fragment, Sequence, FragIndex, FragCount, Nack, NackReason, NextHopFaceId, CachePolicy, CachePolicyType, IncomingFaceId
  • Processing of Fragment body without fragment assembly

NOT implemented (and not planned to be part of https://gerrit.named-data.net/#/c/3254/):

  • for an unknown field that can be ignored, display "unknown field (ignored)"
  • for an unknown field that would cause packet to be dropped, display "unknown field " in red
  • for a field that appears out-of-order, display "out-of-order field " in red
Actions #7

Updated by Alex Afanasyev over 7 years ago

  • Assignee deleted (zipeng wang)
Actions #8

Updated by Junxiao Shi over 7 years ago

  • Status changed from Code review to Feedback

Additional work is needed to fulfill "not implemented" items in note-6.

Actions

Also available in: Atom PDF