Project

General

Profile

Actions

Feature #4189

closed

Remove TcpFactory and UdpFactory prohibited endpoint set

Added by Junxiao Shi almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Faces
Target version:
Start date:
07/17/2017
Due date:
% Done:

100%

Estimated time:
1.50 h

Description

TcpFactory and UdpFactory each has a "prohibited endpoint set" introduced in #1414-4 which prevents creating a face that connects to NFD itself. This was necessary at the time where forwarding strategies were dumb and there was no Nack-Duplicate.
Nowadays, this feature is no longer necessary. Although it still does not make sense to connect to NFD itself, the consequence is less severe: Interest forwarding would trigger Nack-Duplicate, and strategy is expected to handle this situation.
Therefore, "prohibited endpoint set" feature can be removed to reduce complexity of TcpFactory and UdpFactory, especially with the forthcoming dynamic face creation from fine-grained signals.


Related issues 1 (0 open1 closed)

Related to NFD - Feature #4021: FaceSystem: use NetworkMonitor::listNetworkInterfaces()ClosedJunxiao Shi

Actions
Actions #1

Updated by Junxiao Shi almost 7 years ago

  • Related to Feature #4021: FaceSystem: use NetworkMonitor::listNetworkInterfaces() added
Actions #2

Updated by Junxiao Shi almost 7 years ago

  • Subject changed from Removal TcpFactory and UdpFactory prohibited endpoint set to Remove TcpFactory and UdpFactory prohibited endpoint set
  • Assignee set to Junxiao Shi
  • Estimated time set to 1.50 h

This design has been approved on 20170717 NFD call.

Actions #3

Updated by Junxiao Shi over 6 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Junxiao Shi over 6 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 100
Actions #5

Updated by Junxiao Shi over 6 years ago

  • Status changed from Code review to Closed
Actions #6

Updated by Junxiao Shi over 6 years ago

  • Description updated (diff)
Actions

Also available in: Atom PDF