Package: oping
Followup-For: Bug #861856

> When running noping -O foo.csv 127.0.0.1, I get file output like:
> 
> 1493922621,709,"127.0.0.1",11,28
>
> This is somewhat unfortunate, since it's not really comma separated
> values any more.
>
>(My locale has «,» as its decimal separator.)

I am not sure what exactly you consider to be a bug, as you didn't show
an example of what you expected instead.

If it is about the IP, I must remind you that this is NOT a decimal number,
and thus dots are always used (I am not aware of any language that enforce
a different typographical convention that would be here technicaly wrong).


AFAIK, it is fully compliant with RFC-4180 and others.

https://tools.ietf.org/html/rfc4180

My 2nd guess is that you were not aware that you can enclose with a '"'
character the start and the end of any field, even when not strictly
required.  But doing so here is a good idea, as I can imagine securities
issues in some extreme case, perhaps with some unicode domain names or
bogus ones (passed fake domains names that include comas or \n) so as to exploit
scripts that didn't properly sanitized their parameters.

I think I will mark this as WONTFIX, as "fixing" it is likely to break
many scripts that expect those quotes to always be included, and the
current behaviour make sure that scripts are aware that they could have
to handle special strings and characters (it is also much easier that way).

And so I suggest that you or the maintainer simply close this bug.

Have a nice day,

Simon Valiquette


-- System Information:
Debian Release: 9.5
  APT prefers stable-debug
  APT policy: (500, 'stable-debug'), (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 4.9.0-6-686-pae (SMP w/2 CPU cores)
Locale: LANG=fr_CA, LC_CTYPE=fr_CA (charmap=ISO-8859-1), LANGUAGE=fr_CA 
(charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages oping depends on:
ii  libc6         2.24-11+deb9u3
ii  libncursesw5  6.0+20161126-1+deb9u2
ii  liboping0     1.9.0-1+b1
ii  libtinfo5     6.0+20161126-1+deb9u2

oping recommends no packages.

oping suggests no packages.

-- debconf-show failed

Reply via email to