Line 196 of LdapLoginModule.java seems to miss a </dd>?
Rest of webrev look fine.
Valerie
On 3/6/2018 5:38 PM, Weijun Wang wrote:
Please take a review at
http://cr.openjdk.java.net/~weijun/8199154/webrev.00/
<http://cr.openjdk.java.net/%7Eweijun/8199154/webrev.00/>
In LdapLoginModule.java, there seems to be an erroneous DL inside DL.
I removed the outer DL and its incomplete DD. There is no visual
difference after the change.
In Krb5LoginModule.java, there were only DDs and no DT. I've simply
removed all D? tags. The output is little more compact. See the
attached screenshot.
Thanks
Max
