Feature #2699
closedpingserver: allow setting payload size
100%
Description
ndnpingserver uses a fixed string as the response payload.
In certain experiments, it's desirable to reply with a Data packet of a different size.
The payload size should be made adjustable with -s command line option.
Updated by Alex Afanasyev over 10 years ago
+1
Can we replicate options of actual ping command? At least some of them that make sense.
Updated by Davide Pesavento over 10 years ago
Alex Afanasyev wrote:
Can we replicate options of actual ping command? At least some of them that make sense.
Sure, I actually had the same thought. We are already diverging on -t unfortunately (it's -D in ping(8)), but several other options are applicable to ndn-ping as well (at least -AflOqswW).
Updated by Junxiao Shi over 10 years ago
- Description updated (diff)
Can we replicate options of actual ping command? At least some of them that make sense.
Yes, but not in this issue.
I've updated spec to use -s option which matches IP's ping.
Updated by Alex Afanasyev over 10 years ago
On another thought about this. I think we shouldn't have any payload on the server side at all. I think we should simply add a parameter to the client that would add any random number of bytes to the name component. At least this would match IP-based ping, where ping/pongs are basically the same things.
Updated by Junxiao Shi over 10 years ago
I think we should simply add a parameter to the client that would add any random number of bytes to the name component.
ndnping client already has this ability, in the form of "client identifier", although it's not random and it's unnecessary to be random.
At least this would match IP-based ping, where ping/pongs are basically the same things.
NDN Interest is smaller than Data in the common case. Adding a payload allows one to simulate this common case, without having to use the traffic generator.
Updated by Eric Newberry over 10 years ago
- Status changed from New to Code review
Updated by Alex Afanasyev over 10 years ago
OK. Just a thought. I was also thinking to add something to interest to request payload size, but this could become a security hole.
Updated by Eric Newberry over 10 years ago
- Status changed from Code review to Closed
- % Done changed from 0 to 100