Uploaded image for project: 'MariaDB Connector/J'
  1. MariaDB Connector/J
  2. CONJ-342

Empty clientCertificateKeyStoreUrl option handling

Details

    • Bug
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • all
    • 1.5.3
    • Other
    • None

    Description

      Option clientCertificateKeyStoreUrl with empty value (i.e. "") must throw an Exception.
      If set, option clientCertificateKeyStoreUrl must be a valid keystore url value.

      https://github.com/MariaDB/mariadb-connector-j/blob/master/src/main/java/org/mariadb/jdbc/internal/protocol/AbstractConnectProtocol.java#L258

      At the same time, options concerning keystore and truststore like "trustCertificateKeyStoreUrl" are very confusing. It would be better to use java-ish named, and keep current names as alias for compatibility with existing version and mysql. (option "trustCertificateKeyStoreUrl" may be "trustStoreUrl" or "trustStore")

      Attachments

        Activity

          diego dupin Diego Dupin created issue -
          diego dupin Diego Dupin made changes -
          Field Original Value New Value
          Description Option clientCertificateKeyStoreUrl with empty value (i.e. "") must throw an Exception.
          If set, option clientCertificateKeyStoreUrl must be a valid keystore url value.

          https://github.com/MariaDB/mariadb-connector-j/blob/master/src/main/java/org/mariadb/jdbc/internal/protocol/AbstractConnectProtocol.java#L258
          Option clientCertificateKeyStoreUrl with empty value (i.e. "") must throw an Exception.
          If set, option clientCertificateKeyStoreUrl must be a valid keystore url value.

          https://github.com/MariaDB/mariadb-connector-j/blob/master/src/main/java/org/mariadb/jdbc/internal/protocol/AbstractConnectProtocol.java#L258

          At the same time, options concerning keystore and truststore like "trustCertificateKeyStoreUrl" are very confusing. It would be better to use java-ish named, and keep current names as alias for compatibility with existing version and mysql. (option "trustCertificateKeyStoreUrl" may be "trustStoreUrl" or "trustStore")


          diego dupin Diego Dupin made changes -
          Fix Version/s 1.5.3 [ 22300 ]
          diego dupin Diego Dupin made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          diego dupin Diego Dupin made changes -
          Component/s Other [ 12201 ]
          Resolution Fixed [ 1 ]
          Status In Progress [ 3 ] Closed [ 6 ]
          serg Sergei Golubchik made changes -
          Workflow MariaDB v3 [ 76889 ] MariaDB v4 [ 134900 ]

          People

            diego dupin Diego Dupin
            diego dupin Diego Dupin
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Git Integration

                Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.