Small refactoring for loading the certificate

[Hakky54]

Hello!

I noticed the following snippet:

try (InputStream inputStream = getTlsCAFileInputStream()) {
    if (inputStream != null) {
        final X509Certificate certificate = (X509Certificate) CertificateFactory
                .getInstance("X.509")
                .generateCertificate(inputStream);
        final SSLContext sslContext = SSLFactory.builder()
                .withTrustMaterial(certificate)
                .build()
                .getSslContext();
        builder.context(sslContext);
    }
}

At the following place:

amazon-documentdb-jdbc-driver/common/src/main/java/software/amazon/documentdb/jdbc/DocumentDbConnectionProperties.java

Line 1105 in c41d853

final X509Certificate certificate = (X509Certificate) CertificateFactory

It is a small refactoring but this can be refactored to the following snippet:

try (InputStream inputStream = getTlsCAFileInputStream()) {
    if (inputStream != null) {
        final SSLContext sslContext = SSLFactory.builder()
                .withTrustMaterial(CertificateUtils.loadCertificate(inputStream))
                .build()
                .getSslContext();
        builder.context(sslContext);
    }
}

[birschick-bq]

@Hakky54 Thanks for the suggestion. We’ll try to integrate that change, shortly.

[alexey-temnikov]

@Hakky54, the PR with the suggestion is merged - #271. Thank you again for contributing to the project!

[Hakky54]

Awesome, happy to hear that I was able to contribute to this nice library 🙂