RFR: 8245527: LDAP Cnannel Binding support for Java GSS/Kerberos

[Aleks Efimov]

Hi Sean, Alexey,

Adding @systemProperty doesn't look correct here since the properties 
mentioned in the module-info.java are not system properties, they're 
standard JNDI environment properties and setting them via system 
property with the same name won't have any effect, e.g. "java 
-Dcom.sun.jndi.ldap.tls.cbtype=tls-server-end-point".

Best Regards,
Aleksei


On 14/08/2020 14:26, Alexey Bakhtin wrote:
> OK
>
> Updated for all properties in the module-info.java
> Webrev at: http://cr.openjdk.java.net/~abakhtin/8245527/webrev.v16/
>
> Regards
> Alexey
>
>> On 14 Aug 2020, at 16:18, Sean Mullan <sean.mullan at oracle.com> wrote:
>>
>> On 8/14/20 8:41 AM, Alexey Bakhtin wrote:
>>> Hello Sean,
>>> Thank you for review.
>>> I’ve changed wording and replaced @code with @systemProperty (tested, it works for module-info.java)
>> Thanks. Now that you know it works, I think you should change the other properties documented in that file to @systemProperty to be consistent. I think it is fine to do that as part of this fix.
>>
>> --Sean
>>
>>> Updated webrev at: http://cr.openjdk.java.net/~abakhtin/8245527/webrev.v15/
>>> Regards
>>> Alexey
>>>> On 14 Aug 2020, at 14:50, Sean Mullan <sean.mullan at oracle.com> wrote:
>>>>
>>>> On the property wording, change "for LDAP connection" to "for an LDAP connection".
>>>>
>>>> Also, for the definition of the property, can you use the "@systemProperty" annotation instead of "@code"? Does that work inside the module-info.java file?
>>>>
>>>> I added my name as Reviewer.
>>>>
>>>> --Sean
>>>>
>>>> On 7/30/20 6:14 AM, Daniel Fuchs wrote:
>>>>> Hi Alexey,
>>>>> I have added myself as a reviewer to the CSR [1].
>>>>> It would be good to get someone from security-dev to do the
>>>>> same, and then move the CSR state to "Proposed".
>>>>> best regards,
>>>>> -- daniel
>>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8247311
>>>>> On 30/07/2020 10:17, Alexey Bakhtin wrote:
>>>>>> Gentle ping
>>>>>>
>>>>>> Regards
>>>>>> Alexey