#3352 [RFE] Drop support for CSV
Closed: Fixed None Opened 11 years ago by rcritten.

We support CSV in some arguments so one can do things like this:

ipa permission-add --permissions=read,write --attrs=sn,gn ...

This introduces great difficulty when trying to include a value with an embedded space in it. Trying to escape the space in a way that we can parse and make bash happy can be very difficult.

The level of effort in working around the CSV problems has come to the point where the benefits of it are outweighed by the problems.

There are several ways to workaround lack of CSV:

  1. Provide an agugment multiple times on the command-line:

    ipa permission-add --permissions=read --permissions=write ...

  2. Let bash do the expansion for you:

    ipa permission-add --permissions={read,write} ...


I'll start looking into this.

CLI specific, no UXD input needed

CSV was removed from CLI, help was amended:

master:[[BR]]
f2f1ed6 Update plugin docstrings (topic help) to reflect dropped CSV support[[BR]]
5752b35 Update argument docs to reflect dropped CSV support[[BR]]
3a96cbc Drop support for CSV in the CLI client[[BR]]
b4915bd Remove csv_separator and csv_skipspace Param arguments

What about the proposed warnings? They depend on using JSON as our CLI transport protocol and it may not happen this release. Thus, we may want to create an own ticket for this piece and close this ticket.

Warning RFE filed as #3456, closing this ticket.

Metadata Update from @rcritten:
- Issue assigned to pviktori
- Issue set to the milestone: FreeIPA 3.2 - 2013/02

7 years ago

Login to comment on this ticket.

Metadata