Project

General

Profile

Actions

Bug #4926

closed

Data.wireDecode is possibly forcing the presence of metainfo tlv block

Added by Edward Lu almost 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Start date:
05/05/2019
Due date:
% Done:

0%

Estimated time:

Description

Hello,

I have been trying to decode a data packet encoded by the ndn-lite library, and it seems like the wire decoding of the jndn library assumes that a metainfo tlv block has to be present, even though it is optional in the Data packet specification (https://named-data.net/doc/NDN-packet-spec/current/data.html). Because of this, I get an error when trying to decode a wire encoded data packet that does not contain a metainfo tlv block.

I feel like this is likely some mistake on my part, but I cannot find where the correct data decoding function I should use to ignore these optional tlv blocks is.

I have attached screenshots of the code and output.


Files

Actions #1

Updated by Anonymous almost 5 years ago

  • Project changed from jndn to NDN-CCL
  • Assignee set to Anonymous

Hi Edward. Yes, this is a bug. I moved it to the NDN-CCL projects because it affects all Common Client Libraries. I'll fix it first in jNDN and let you know.

Actions #2

Updated by Anonymous almost 5 years ago

  • Status changed from New to Feedback

@Edward Lu I pushed the fix to GitHub. Are you able to pull the latest from GitHub and test, or do you need Maven packages?

Actions #3

Updated by Edward Lu almost 5 years ago

Ah thank you. I think I am using the Maven packages in Android Studio (I have a dependency on net.named-data:jndn-android:0.21 in my Project Structure); would that require me to use Maven packages, or should I find a way to update it in Android studio by pulling the latest version from github?

Actions #4

Updated by Anonymous almost 5 years ago

I pushed version 0.22, so you can test the Maven package.

https://search.maven.org/search?q=g:net.named-data

Actions #5

Updated by Edward Lu almost 5 years ago

Ah thank you; I tested it and it no longer forces the presence of the metainfo block.

Actions #6

Updated by Anonymous almost 5 years ago

Thank you for testing. I will update the other CCL libraries.

Actions #7

Updated by Anonymous almost 5 years ago

  • Status changed from Feedback to Closed

Changes pushed to master in all libraries.

Actions

Also available in: Atom PDF