You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@guacamole.apache.org by GitBox <gi...@apache.org> on 2020/06/25 22:51:44 UTC
[GitHub] [guacamole-manual] necouchman opened a new pull request #144: GUACAMOLE-728: Update JDBC SSL mode documentation to reflect updated default
necouchman opened a new pull request #144:
URL: https://github.com/apache/guacamole-manual/pull/144
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [guacamole-manual] mike-jumper merged pull request #144: GUACAMOLE-728: Update JDBC SSL mode documentation
Posted by GitBox <gi...@apache.org>.
mike-jumper merged pull request #144:
URL: https://github.com/apache/guacamole-manual/pull/144
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [guacamole-manual] mike-jumper commented on a change in pull request #144: GUACAMOLE-728: Update JDBC SSL mode documentation
Posted by GitBox <gi...@apache.org>.
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:
users@infra.apache.org
[GitHub] [guacamole-manual] necouchman commented on pull request #144: GUACAMOLE-728: Update JDBC SSL mode documentation
Posted by GitBox <gi...@apache.org>.
necouchman commented on pull request #144:
URL: https://github.com/apache/guacamole-manual/pull/144#issuecomment-649889049
I believe I did get the Postgres default update in there - down toward the bottom?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [guacamole-manual] mike-jumper commented on pull request #144: GUACAMOLE-728: Update JDBC SSL mode documentation
Posted by GitBox <gi...@apache.org>.
mike-jumper commented on pull request #144:
URL: https://github.com/apache/guacamole-manual/pull/144#issuecomment-649889999
Ah, you're right. I just needed to click the arrows a bit.
----------------------------------------------------------------
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:
users@infra.apache.org