Project

General

Profile

Actions

Feature #3522

closed

FaceUri syntax for netdev-bound faces

Added by Junxiao Shi about 8 years ago. Updated over 5 years ago.

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

100%

Estimated time:
2.00 h

Description

Define FaceUri syntax for netdev-bound faces (#3521).

Update FaceUri parser to accept this syntax, which will be used in face dataset.


Related issues 1 (1 open0 closed)

Blocks NFD - Feature #3521: Netdev-bound facesIn Progress

Actions
Actions #1

Updated by Junxiao Shi about 8 years ago

Actions #2

Updated by Davide Pesavento about 8 years ago

This task should be moved to the ndn-cxx project.

Actions #3

Updated by Junxiao Shi about 8 years ago

  • Project changed from NFD to ndn-cxx
  • Category changed from Faces to Utils
  • Target version changed from v0.5 to v0.5

FaceMgmt revision 51 defines the syntax.

Actions #4

Updated by Davide Pesavento about 8 years ago

There's a small problem with the "%" sign, as I pointed out here http://redmine.named-data.net/issues/1428#note-4, and the percent-encoded version would be kind of ugly and harder to read in my opinion.

What were the other proposals for the syntax?

Actions #5

Updated by Davide Pesavento almost 8 years ago

  • Assignee deleted (Andrea Tosatto)
  • Start date deleted (03/08/2016)
Actions #6

Updated by Junxiao Shi over 7 years ago

  • Assignee set to Weiwei Liu
  • Target version changed from v0.5 to v0.6
Actions #7

Updated by Weiwei Liu over 7 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Jeff Burke over 7 years ago

Needs to target v0.5 of dependent Feature #3521.

Actions #9

Updated by Davide Pesavento over 7 years ago

Jeff Burke wrote:

Needs to target v0.5 of dependent Feature #3521.

v0.5 has already been released, so v0.6 is the correct target.

Actions #10

Updated by Junxiao Shi over 7 years ago

FaceMgmt revision 59 adds %25 to the syntax.

Actions #11

Updated by Junxiao Shi over 7 years ago

  • Estimated time set to 2.00 h
Actions #12

Updated by Weiwei Liu over 7 years ago

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

Updated by Junxiao Shi over 7 years ago

when #1428 is eventually implemented, we might have a problem here... how do we distinguish the two?

How about changing the syntax to udp4+dev://eth1:6363?

Actions #14

Updated by Davide Pesavento over 7 years ago

Junxiao Shi wrote:

How about changing the syntax to udp4+dev://eth1:6363?

Fine with me.

Actions #15

Updated by Junxiao Shi over 7 years ago

FaceMgmt rev60 has the new syntax.

It's worth noting that the <port> part is required, and such FaceUris are always canonical.

Actions #16

Updated by Junxiao Shi over 7 years ago

can't we add a default port during canonicalization?

There isn't a valid use case for that.
FaceUri for NIC-associated permanent faces are never intended to be entered by an operator. It can only be returned as the LocalUri in a faces/list or faces/query dataset. Therefore, they will always be constructed as canonical.

Actions #17

Updated by Davide Pesavento over 7 years ago

Junxiao Shi wrote:

can't we add a default port during canonicalization?

There isn't a valid use case for that.
FaceUri for NIC-associated permanent faces are never intended to be entered by an operator. It can only be returned as the LocalUri in a faces/list or faces/query dataset. Therefore, they will always be constructed as canonical.

I know that. However there are things in FaceUri that are a little more generic and broader than the bare minimum required by NFD. For example there's a constructor taking an arbitrary scheme, even though we only need it for ws:// and wsclient://... Anyway, not a big deal, it can always be changed later.

Actions #18

Updated by Junxiao Shi over 7 years ago

  • Status changed from Code review to Closed
  • % Done changed from 0 to 100
Actions #19

Updated by Davide Pesavento over 5 years ago

  • Subject changed from FaceUri syntax for NIC-associated permanent faces to FaceUri syntax for netdev-bound faces
  • Description updated (diff)
Actions

Also available in: Atom PDF