Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

TrustManager not set in AppConfig class in newRestDatabaseClientConfig(int port) and newAppServicesDatabaseClient(String databaseName) #294

Closed
srinathgit opened this issue Aug 3, 2018 · 2 comments
Assignees
Milestone

Comments

@srinathgit
Copy link

TrustManager is not set in public DatabaseClientConfig newRestDatabaseClientConfig(int port) and public DatabaseClient newAppServicesDatabaseClient(String databaseName) .

something like

    public DatabaseClientConfig newRestDatabaseClientConfig(int port) {
	    DatabaseClientConfig config = new DatabaseClientConfig(getHost(), port, getRestAdminUsername(), getRestAdminPassword());
	    config.setSecurityContextType(restSecurityContextType);
	    config.setSslHostnameVerifier(getRestSslHostnameVerifier());
	    config.setSslContext(getRestSslContext());
	    config.setCertFile(getRestCertFile());
	    config.setCertPassword(getRestCertPassword());
	    config.setExternalName(getRestExternalName());
	    **config.setTrustManager(getRestTrustManager());**
	    return config;
    }
@srinathgit srinathgit changed the title TrustManager not set in AppConfig class TrustManager not set in AppConfig class in newRestDatabaseClientConfig(int port) and newAppServicesDatabaseClient(String databaseName) Aug 3, 2018
@rjrudin
Copy link
Contributor

rjrudin commented Aug 6, 2018

@srinathgit Just to be clear - you're asking for appConfig.restTrustManager to be added and then used in this method, correct?

@srinathgit
Copy link
Author

Yes, in addition appConfig.appServicesTrustManager also probably has to be added and used in public DatabaseClient newAppServicesDatabaseClient(String databaseName) method

@rjrudin rjrudin added this to the 3.8.3 milestone Aug 7, 2018
@rjrudin rjrudin self-assigned this Aug 7, 2018
rjrudin added a commit that referenced this issue Aug 8, 2018
…nager

#294 Can now set TrustManager for REST and AppServices servers
@rjrudin rjrudin closed this as completed Aug 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants