[GitHub] flink pull request #6312: [FLINK-9792] Added custom Description class for Co...

[dawidwys]

Github user dawidwys commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6312#discussion_r201796123
  
    --- Diff: docs/_includes/generated/security_configuration.html ---
    @@ -10,7 +10,7 @@
             <tr>
                 <td><h5>security.ssl.algorithms</h5></td>
                 <td style="word-wrap: 
break-word;">"TLS_DHE_RSA_WITH_AES_128_GCM_SHA256,TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256,TLS_DHE_RSA_WITH_AES_256_GCM_SHA384,TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384"</td>
    -            <td>The comma separated list of standard SSL algorithms to be 
supported. Read more &#60;a 
href="http://docs.oracle.com/javase/8/docs/technotes/guides/security/StandardNames.html#ciphersuites"&#62;here&#60;/a&#62;.</td>
    +            <td>The comma separated list of standard SSL algorithms to be 
supported. Read more &lt;a 
href="http://docs.oracle.com/javase/8/docs/technotes/guides/security/StandardNames.html#ciphersuites"&gt;here&lt;/a&gt;.</td>
    --- End diff --
    
    Why do you think so? &gt; and &lt; are valid html encodings(I think it is 
even more common way of escaping < and >). I've changed it, cause it was 
impossible to retrieve the other version from `Jsoup` which is used in tests. 
I've also checked it renders well both in Chrome as well as in Firefox.


---