Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-06-02 Thread Petr Vobornik
On 05/31/2016 03:57 PM, Nathaniel McCallum wrote: > On Tue, 2016-05-31 at 15:25 +0200, Petr Vobornik wrote: >> On 05/31/2016 02:49 PM, Nathaniel McCallum wrote: >>> On Mon, 2016-05-30 at 19:08 +0300, Alexander Bokovoy wrote: On Mon, 30 May 2016, Petr Vobornik wrote: > On 05/27/2016 06:00

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-31 Thread Nathaniel McCallum
On Tue, 2016-05-31 at 15:25 +0200, Petr Vobornik wrote: > On 05/31/2016 02:49 PM, Nathaniel McCallum wrote: > > On Mon, 2016-05-30 at 19:08 +0300, Alexander Bokovoy wrote: > > > On Mon, 30 May 2016, Petr Vobornik wrote: > > > > On 05/27/2016 06:00 PM, Nathaniel McCallum wrote: > > > > > Pavel,

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-31 Thread Petr Vobornik
On 05/31/2016 02:49 PM, Nathaniel McCallum wrote: > On Mon, 2016-05-30 at 19:08 +0300, Alexander Bokovoy wrote: >> On Mon, 30 May 2016, Petr Vobornik wrote: >>> On 05/27/2016 06:00 PM, Nathaniel McCallum wrote: Pavel, since we made the change here from a StrEnum to a Str, we need to

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-31 Thread Nathaniel McCallum
On Mon, 2016-05-30 at 19:08 +0300, Alexander Bokovoy wrote: > On Mon, 30 May 2016, Petr Vobornik wrote: > > On 05/27/2016 06:00 PM, Nathaniel McCallum wrote: > > > Pavel, since we made the change here from a StrEnum to a Str, we > > > need > > > to update the UI patch accordingly. > > > > How

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-30 Thread Nathaniel McCallum
On Mon, 2016-05-30 at 19:08 +0300, Alexander Bokovoy wrote: > On Mon, 30 May 2016, Petr Vobornik wrote: > > On 05/27/2016 06:00 PM, Nathaniel McCallum wrote: > > > Pavel, since we made the change here from a StrEnum to a Str, we > > > need > > > to update the UI patch accordingly. > > > > How

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-30 Thread Alexander Bokovoy
On Mon, 30 May 2016, Petr Vobornik wrote: On 05/27/2016 06:00 PM, Nathaniel McCallum wrote: Pavel, since we made the change here from a StrEnum to a Str, we need to update the UI patch accordingly. How should admin know what to write there intuitively? Shouldn't Web UI or CLI advertise the

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-30 Thread Petr Vobornik
On 05/27/2016 06:00 PM, Nathaniel McCallum wrote: > Pavel, since we made the change here from a StrEnum to a Str, we need > to update the UI patch accordingly. How should admin know what to write there intuitively? Shouldn't Web UI or CLI advertise the indicators supported by IPA? E.g. CLI in

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-27 Thread Nathaniel McCallum
Pavel, since we made the change here from a StrEnum to a Str, we need to update the UI patch accordingly. On Fri, 2016-05-27 at 11:55 -0400, Nathaniel McCallum wrote: > On Fri, 2016-05-27 at 18:35 +0300, Alexander Bokovoy wrote: > > On Fri, 27 May 2016, Nathaniel McCallum wrote: > > > All core

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-27 Thread Nathaniel McCallum
On Fri, 2016-05-27 at 18:35 +0300, Alexander Bokovoy wrote: > On Fri, 27 May 2016, Nathaniel McCallum wrote: > > All core functionality for authentication indicators has already > > been > > merged. All that is left is the CLI and UI patches. Attached is the > > CLI > > patch. > > > > One

Re: [Freeipa-devel] [PATCH 0093] Enable service authentication indicator management

2016-05-27 Thread Alexander Bokovoy
On Fri, 27 May 2016, Nathaniel McCallum wrote: All core functionality for authentication indicators has already been merged. All that is left is the CLI and UI patches. Attached is the CLI patch. One outstanding question that I have is how to future-proof this patch. Right now, we want to only