Project

General

Profile

Actions

Feature #2189

closed

Pick Transport type with client.conf

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:
3.00 h

Description

Add a transport option to client.conf.

Its value is a FaceUri that represents a listener. This FaceUri does not have to be canonical.

To pick UnixTransport: transport=unix:///var/run/nfd.sock

Path can be changed if necessary.

To pick TcpTransport: transport=tcp://localhost:6363

Port can be changed if necessary.
"tcp://localhost" is recommended; other variants such as "tcp4://127.0.0.1" or "tcp6://[::1]" is acceptable but not recommended.
Connecting to a non-local endpoint is unsupported but not forbidden.

The default constructor of Face class should respect this configuration, instead of always creating UnixTransport.


Related issues 2 (0 open2 closed)

Related to NFD - Bug #2360: Multiple scenarios: ERROR: error while connecting to the forwarder (No such file or directory)ClosedHila Ben Abraham01/08/2015

Actions
Blocks NFD - Bug #2190: NRD cannot start when NFD disables UNIX facesClosed

Actions
Actions #1

Updated by Junxiao Shi over 9 years ago

  • Related to Bug #2188: FaceManager creates UnixStreamChannel when listen=no added
Actions #2

Updated by Junxiao Shi over 9 years ago

  • Blocks Bug #2190: NRD cannot start when NFD disables UNIX faces added
Actions #3

Updated by Junxiao Shi over 9 years ago

  • Related to deleted (Bug #2188: FaceManager creates UnixStreamChannel when listen=no)
Actions #4

Updated by Anonymous over 9 years ago

  • Status changed from New to In Progress
  • Assignee set to Anonymous
Actions #5

Updated by Junxiao Shi over 9 years ago

I want to point out that this Feature lacks design approval. If you go implement it, there's risk of having to rework everything in case design is rejected.

Actions #6

Updated by Anonymous over 9 years ago

Understood.

Actions #7

Updated by Anonymous over 9 years ago

What do you thinking of changing to use Face URIs to specify the protocol, host, and port? Handling Unix transport without a Face URI is trivial, but I'd hate to replicate existing functionality to determine whether a TCP/UDP host option was IPv4, v6, or a hostname.

To pick UnixTransport:

transport=unix:///var/run/nfd.sock

To pick IPv4 TcpTransport:

transport=tcp4://localhost:6363
Actions #8

Updated by Junxiao Shi over 9 years ago

  • Description updated (diff)

I agree with the design in note-7.

Actions #9

Updated by Alex Afanasyev over 9 years ago

ok. I agree with the design.

Actions #10

Updated by Anonymous over 9 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 70
Actions #11

Updated by Anonymous over 9 years ago

Where can I find the ndn-cxx documentation that needs to be updated? Is it a separate repo or something under docs?

Actions #12

Updated by Alex Afanasyev over 9 years ago

  • Status changed from Code review to In Progress
Actions #13

Updated by Junxiao Shi over 9 years ago

Where can I find the ndn-cxx documentation that needs to be updated? Is it a separate repo or something under docs?

It's a separate repository. http://www.lists.cs.ucla.edu/pipermail/nfd-dev/2014-December/000673.html

Actions #14

Updated by Anonymous over 9 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 70 to 100

Added text to internals describing client.conf and comments are appreciated. I will close the issue in a few days if there are no suggestions.

Actions #15

Updated by Junxiao Shi over 9 years ago

  • Status changed from Code review to Closed

I have reviewed ndn-cxx-docs commit f6d632cc8b923a3e6c497d463c6a93a7b45607b8.

Actions #16

Updated by Junxiao Shi over 9 years ago

  • Related to Bug #2360: Multiple scenarios: ERROR: error while connecting to the forwarder (No such file or directory) added
Actions

Also available in: Atom PDF