<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 08/19/2014 10:46 PM, Nathaniel
McCallum wrote:<br>
</div>
<blockquote cite="mid:1408481179.3505.6.camel@redhat.com"
type="cite">
<pre wrap="">Also, remove the attempt to load the objectClasses when absent. This
never makes sense during an add operation.
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/4455">https://fedorahosted.org/freeipa/ticket/4455</a>
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Freeipa-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre>
</blockquote>
<font face="Times New Roman, Times, serif">Hello Nathaniel,<br>
<br>
</font>
<blockquote><font face="Times New Roman, Times, serif">Reading the
patch I have one novice remark. In the previous code,
'objectclass' was added to 'entry_attr' in the case it was
missing in 'entry_attr' (at the condition
'ipatokenradiusconfiglink' was defined). In the new code, if
'objectclass' is missing it is not added. Is it ok ?<br>
<br>
Also, regarding the 'user life cycle'. Staging users are candidate
to become Active users. I wonder if Staging users should also
contain your fix that add the ipaUserAuthTypeClass.</font><br>
<br>
<font face="Times New Roman, Times, serif">thanks<br>
thierry<br>
</font></blockquote>
</body>
</html>