[Freeipa-devel] [PATCH 0078] Enable QR code display by default in otptoken-add

[Simo Sorce]

On Fri, 14 Nov 2014 14:08:24 +0100
Petr Viktorin <pviktori at redhat.com> wrote:

> On 11/14/2014 01:18 PM, Petr Vobornik wrote:
> [...]
> >>
> >> Nope, defaults are filled in by the client. (And also on the
> >> server if they're still missing; it's part of the common
> >> validation.)
> >
> > IMHO this is quite unfortunate behavior which may also fail
> > horribly if there is a newer client and an older server ->
> > backwards compatibility is on API level, not CLI level. Defaults
> > should be filled by server, not a client.  We should seriously
> > reconsider the design of our CLI. But that's for different, future
> > discussion.
> 
> You can't use a newer client with an older server, you get a 
> VersionError in that case.

Does it break only for this command ?
Or in general.
If a Fedora 21 client can't talk to a RHEL 6 server we have a huge
problem that we need to fix *yesterday*.

> Feel free to file a ticket. But yes, redesigning the API is not
> exactly a priority.
> 
> > That's said and given the circumstances, it is easier and cleaner to
> > return the --qrcode back as no_param now than to deal with potential
> > future issues.
> 
> What's the reason to break the CLI by making it no_param?
> 
> 



-- 
Simo Sorce * Red Hat, Inc * New York