Project

General

Profile

Actions
Copy link

Bug #2031

closed

fib/add-nexthop: FaceId should be optional

Added by Tai-Lin Chu over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tai-Lin Chu
Category:
Management
Target version:
v0.3
Start date:
10/06/2014
Due date:
% Done:

100%

Estimated time:
1.00 h

Description

Steps to reproduce:

  1. send a fib/add-nexthop command without FaceId field in the parameter.

Expected: FaceId is implied as the requesting Face.

Actual: 400 error

Actions
Copy link
 #1

Updated by Junxiao Shi over 9 years ago

Please provide a code snippet to reproduce this issue.

Actions
Copy link
 #2

Updated by Tai-Lin Chu over 9 years ago

fib add-nexthop?

I actually find the bug. It is pretty obvious.

https://github.com/named-data/NFD/blob/68bc1e0dce68c0a0ef98fe5e3fee15a4d8b21fc8/daemon/mgmt/fib-manager.cpp#L146

that !parameters.hasFaceId()

Actions
Copy link
 #3

Updated by Junxiao Shi over 9 years ago

  • Subject changed from face id cannot be ignored to fib/add-nexthop: FaceId should be optional
  • Description updated (diff)
  • Category set to Management
  • Target version set to v0.3
  • Estimated time set to 1.00 h
Actions
Copy link
 #4

Updated by Tai-Lin Chu over 9 years ago

https://github.com/named-data/NFD/commit/7564d97d135b3fc9bb1150cdfb7505177813f572

The origin commit that introduces the bug. I guess it is not a bug back then.

Actions
Copy link
 #5

Updated by Junxiao Shi over 9 years ago

  • Assignee set to Tai-Lin Chu

Please cancel GitHub pull request, and submit the patch to Gerrit for review.

Actions
Copy link
 #6

Updated by Junxiao Shi over 9 years ago

The face ID is optional only for self registrations, not for any other fib operations to add routes.

No. fib/add-nexthop command allows optional FaceId.

Actions
Copy link
 #7

Updated by Lan Wang over 9 years ago

In what situation is the optional faceID needed? If there has not been any problems so far, I assume nrd always supplies the faceID.

Actions
Copy link
 #8

Updated by Junxiao Shi over 9 years ago

The protocol allows omitting FaceId. The default value of FaceId is 0, which means "the requesting face".

nrd always setting FaceId doesn't mean FibManager can violate protocol.

Actions
Copy link
 #9

Updated by Lan Wang over 9 years ago

I was asking why the protocol defines the FaceID as optional. If I understand correctly, the fib commands are issued only by nrd. Applications do not directly interact with nfd to register their prefixes, but the optional FaceID is useful only in this case.

Actions
Copy link
 #10

Updated by Junxiao Shi over 9 years ago

The recommended way to represent "self" is omitting the FaceId field. This is consistent with other commands such as faces/enable-local-control and rib/register.

Allowing FaceId=0 is due to historical reason.

Actions
Copy link
 #11

Updated by Alex Afanasyev over 9 years ago

  • Status changed from New to Code review
Actions
Copy link
 #12

Updated by Junxiao Shi over 9 years ago

  • Status changed from Code review to Closed
  • % Done changed from 0 to 100
Actions
Copy link

Also available in: Atom PDF