<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 1, 2015 at 5:32 PM, Nate Klingenstein <span dir="ltr"><<a href="mailto:ndk@internet2.edu" target="_blank">ndk@internet2.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">> But I suppose we could support some velocity syntax there if people thought that was valuable.<br>
<br>
</span>It’s not worth a huge time investment and it’s not near the top of my wishlist.  It was just something that caught me off guard, and I’ll bet it’s caught others, since it’s the odd property out.  It’d also be neat if the properties were both named searchFilter or userFilter if they have the same semantics, but they might not.<br>
<span class=""><br></span></blockquote><div><br></div><div>Created <a href="https://issues.shibboleth.net/jira/browse/IDP-827">https://issues.shibboleth.net/jira/browse/IDP-827</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">
> The team made a decision to use startTLS by default for all LDAP integration. The support for this feature is pretty broad at this point. The ':true' spring syntax is just defaulting the value if that property isn't set.<br>
<br>
</span>That makes sense.  Is there a reason why the defaulting here is done with Spring versus somewhere else like most of the others?  Just for the composability?  Are there a lot more?  Is it worth making the defaulting mechanism consistent?  I have no qualms with the shorthand or the convention at all, but it would be nice to ooze towards consistency with an explanation in a comment.<br>
<br>
aithon:conf ndk$ grep -R \:true\} *<br>
attribute-resolver-full.xml:        useStartTLS="%{idp.attribute.resolver.LDAP.useStartTLS:true}"><br>
attribute-resolver-ldap.xml:        useStartTLS="%{idp.attribute.resolver.LDAP.useStartTLS:true}"><br>
authn/ldap-authn-config.xml:        p:useStartTLS="%{idp.authn.LDAP.useStartTLS:true}"<br>
authn/ldap-authn-config.xml:        p:validatePeriodically="%{idp.pool.LDAP.validatePeriodically:true}"<br>
ldap.properties:idp.attribute.resolver.LDAP.useStartTLS         = %{idp.authn.LDAP.useStartTLS:true}<br></blockquote><div><br></div><div>and <a href="https://issues.shibboleth.net/jira/browse/IDP-826">https://issues.shibboleth.net/jira/browse/IDP-826</a></div><div><br></div><div>--Daniel Fisher</div><div><br></div></div></div></div>