[Freeipa-devel] [PATCHES] 0183-0185 Drop CSV support
Martin Kosek
mkosek at redhat.com
Fri Feb 22 13:01:22 UTC 2013
On 02/21/2013 06:08 PM, Petr Viktorin wrote:
> These patches remove CSV parsing from the client.
>
> Ticket: https://fedorahosted.org/freeipa/ticket/3352
> Design: http://freeipa.org/page/V3/Drop_CSV
>
>
> The design page also talks about adding a warning for the user when they seem
> to use CSV, but this will need the JSON transport so it's not included in these
> patches.
>
> The "csv" flag is left on parameters so when that warning is added, we know for
> which params to show it.
>
Generally, this works fine. This finally allows us to easily add records with
quote or comma:
# ipa dnsrecord-add example.com txt1 --txt-rec="foo, bar"
Record name: txt1
TXT record: foo, bar
# ipa dnsrecord-add example.com txt1 --txt-rec='"bar,bar"'
Record name: txt1
TXT record: foo, bar, "bar,bar"
# ipa dnsrecord-show example.com txt1 --all --raw
dn: idnsname=txt1,idnsname=example.com,cn=dns,dc=example,dc=com
idnsname: txt1
txtrecord: foo, bar
txtrecord: "bar,bar"
objectclass: top
objectclass: idnsrecord
Just couple minor remarks:
1) Can we then update the csv flag description? It is now a bit misleading
given its new function
- csv: this multivalue attribute is given in CSV format
2) We need to also update our help, for example selfservice online help
presents a CSV formatted option use:
ipa selfservice-add --permissions=write --attrs=street,postalCode,l,c,st
"Users manage their own address"
Btw if we suggest the curly braces shortcut, we may also want to write that it
applies for BASH shell only...
Few other examples I saw:
ipa group-add-member --users=test1,test2 localadmins
Martin
More information about the Freeipa-devel
mailing list