Add support for machine readable output

Question

Add support for machine readable output

dgzlopes opened this issue 4 years ago · 7 comments

dgzlopes commented 4 years ago

Add a flag, where you can specify the desired format.

e.g --output json or --output xml

Moras-del commented 4 years ago

okay :)

Answer 1 · 2020-10-01T12:11:36.000Z

I can solve that.

Answer 2 · 2020-10-01T12:19:50.000Z

Nice @Moras-del! If you need help, feel free to ping me.

Answer 3 · 2020-10-01T12:41:44.000Z

I would like to be sure. Do You want to support xml/json format only when program is used as module? E.g def measure_latency( host, port = 443, timeout = 5, runs = 1, wait = 1, human_output = False, format='json' ) -> list:
as I understand the content returned from the CLI should also be returned from the function call (when human_output=False, format='json/xml/None') in specified format, like:
'host': 'www.google.com', 'protocol': 'tcp', 'seq': '1', 'port': '443', 'timeout': '5', 'time': '200ms'
Am I correct?

Answer 4 · 2020-10-01T13:08:50.000Z

Yep. So we can use tools like jq to work with the data.

Human-output should be the default, but I think having other options (e.g JSON, XML, CSV, whatever) might be useful.

Answer 5 · 2020-10-01T13:24:02.000Z

Ok but measure_latency() should also return list of json objects instead of latency points if specified? Or format should be used ONLY for working with CLI?

Answer 6 · 2020-10-01T13:31:17.000Z

In my opinion: CLI only 😛

I don't want to mess with how people invoke measure_latency(). And I don't think it's useful to get a list of JSON objects!