Project

General

Profile

Actions

Task #3137

closed

ping: Refactor code to allow integration with ndnSIM

Added by Alex Afanasyev about 10 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Normal
Start date:
08/26/2015
Due date:
% Done:

100%

Estimated time:

Description

There are a few places that require small refactoring of client::Ping and server::PingServer:

  • only owner of ping/server classes should manage async thread (no calls to io_service.run/stop)
  • only owner of server should own KeyChain (this way ndnSIM can substitute KeyChain instance)
  • ping/server classes should gracefully handle termination. Right now, both ping and server don't clean up states and simply rely on terminating of face operations and even using exit(1) in some cases. To run experimentation within ndnSIM, stopping has to be done in a graceful way, with properly cancelling all events and unregistering prefixes.

Files

20150827192608.tgz (151 KB) 20150827192608.tgz integ 2392,4 Junxiao Shi, 08/27/2015 03:17 PM
20150827224855.tgz (152 KB) 20150827224855.tgz Alex Afanasyev, 08/27/2015 09:01 PM
20150831095442.tgz (150 KB) 20150831095442.tgz integ 2392,6 Junxiao Shi, 08/31/2015 03:59 AM
20150831163044.tgz (151 KB) 20150831163044.tgz Alex Afanasyev, 08/31/2015 11:25 AM
Actions #2

Updated by Alex Afanasyev about 10 years ago

Integration tests are passing now.

Actions #5

Updated by Alex Afanasyev about 10 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF