mike-jumper commented on a change in pull request #144:
URL: https://github.com/apache/guacamole-manual/pull/144#discussion_r445911576



##########
File path: src/chapters/jdbc-auth.xml
##########
@@ -100,7 +100,13 @@
                     </para>
                     <para>The MySQL Driver implements several parameters 
specific to configuring
                         SSL for secure connections to MySQL servers that 
support or require
-                        encrypted communications.  The parameters are as 
follows:</para>
+                        encrypted communications. The SSL configuration 
options are targeted
+                        at newer versions of the MySQL Connector/J and MariaDB 
Connector/J.
+                        Older versios of MySQL Connector/J have known issues 
with SSL

Review comment:
       versions*
   
   For the sake of readability, I would additionally:
   
   * Wrap that main "older versions have known issues" bit within `<emphasis>`, 
as it's fairly important
   * Clean up the repeated "Connector/J" a bit, as it's hard to read with all 
those slashes and repetition.
   
    For example:
   
   > The MySQL Driver implements several parameters specific to configuring SSL 
for secure connections to MySQL servers that support or require encrypted 
communications. _Older versions of Connector/J driver have known issues with 
SSL verification_. If you experience problems connecting to SSL-secured MySQL 
or MariaDB databases, it is recommended that you update to a current version of 
the driver.
   >
   > Configuration parameters for MySQL-compatible SSL support are as follows:




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to