go-ndn / nfd

NDN forwarder
GNU General Public License v2.0
6 stars 0 forks source link

Prefix registration is incompatible with ndn-cxx #4

Closed yoursunny closed 9 years ago

yoursunny commented 9 years ago

go-nfd accepts prefix registration commands under /localhop/nfd command prefix only. ndn-cxx and other libraries by default send registration command using /localhost/nfd command prefix. go-nfd should accept both command prefixes in order to be compatible with those libraries.

taylorchu commented 9 years ago

https://github.com/go-ndn/nfd/commit/b0055160a5527d8c512bb8fc54fea1e3b2a178ad

yoursunny commented 9 years ago

The solution still appears incorrect. ndn-tlv-peek /localhost/nfd/rib/list times out.

go-nfd debug log says:

[127.0.0.1:44785] face created
[core] /rib/list
[127.0.0.1:44785] face removed

Reply Data Name is probably wrong.

taylorchu commented 9 years ago

You can try one more time now. It should be fixed.

On Thu, May 14, 2015 at 7:02 PM, yoursunny notifications@github.com wrote:

The solution still appears incorrect. ndn-tlv-peek /localhost/nfd/rib/list times out.

go-nfd debug log says:

[127.0.0.1:44785] face created [core] /rib/list [127.0.0.1:44785] face removed

Reply Data Name is probably wrong.

— Reply to this email directly or view it on GitHub https://github.com/go-ndn/nfd/issues/4#issuecomment-102221113.

yoursunny commented 9 years ago

This particular issue is resolved now. I opened #5 for a different but related issue.