Feature #2699
closed
pingserver: allow setting payload size
Added by Junxiao Shi over 10 years ago.
Updated over 10 years ago.
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.
+1
Can we replicate options of actual ping command? At least some of them that make sense.
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).
- 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.
- Assignee set to Eric Newberry
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.
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.
- Status changed from New to Code review
OK. Just a thought. I was also thinking to add something to interest to request payload size, but this could become a security hole.
- Status changed from Code review to Closed
- % Done changed from 0 to 100
Also available in: Atom
PDF