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

Commit b7eb082

Browse files
committed
#243 Removed depreceated authentication properties
1 parent 3ef01e1 commit b7eb082

File tree

3 files changed

+0
-34
lines changed

3 files changed

+0
-34
lines changed

src/main/java/com/marklogic/appdeployer/AppConfig.java

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,6 @@ public class AppConfig {
7777
private String restCertFile;
7878
private String restCertPassword;
7979
private String restExternalName;
80-
@Deprecated
81-
private Authentication restAuthentication;
8280
private Integer restPort = DEFAULT_PORT;
8381
private Integer testRestPort;
8482

@@ -92,8 +90,6 @@ public class AppConfig {
9290
private String appServicesCertFile;
9391
private String appServicesCertPassword;
9492
private String appServicesExternalName;
95-
@Deprecated
96-
private Authentication appServicesAuthentication;
9793

9894
// These can all be set to override the default names that are generated off of the "name" attribute.
9995
private String groupName = DEFAULT_GROUP;
@@ -293,7 +289,6 @@ public DatabaseClient newTestDatabaseClient() {
293289

294290
public DatabaseClientConfig newRestDatabaseClientConfig(int port) {
295291
DatabaseClientConfig config = new DatabaseClientConfig(getHost(), port, getRestAdminUsername(), getRestAdminPassword());
296-
config.setAuthentication(getRestAuthentication());
297292
config.setSecurityContextType(restSecurityContextType);
298293
config.setSslHostnameVerifier(getRestSslHostnameVerifier());
299294
config.setSslContext(getRestSslContext());
@@ -323,7 +318,6 @@ public DatabaseClient newSchemasDatabaseClient() {
323318
public DatabaseClient newAppServicesDatabaseClient(String databaseName) {
324319
DatabaseClientConfig config = new DatabaseClientConfig(getHost(), getAppServicesPort(), getAppServicesUsername(), getAppServicesPassword());
325320
config.setDatabase(databaseName);
326-
config.setAuthentication(getAppServicesAuthentication());
327321
config.setSecurityContextType(appServicesSecurityContextType);
328322
config.setSslHostnameVerifier(getAppServicesSslHostnameVerifier());
329323
config.setSslContext(getAppServicesSslContext());
@@ -521,20 +515,6 @@ public void setGroupName(String groupName) {
521515
this.groupName = groupName;
522516
}
523517

524-
/**
525-
* @return the MarkLogic Java Client {@code Authentication} object that is used for authenticating with a REST API
526-
* server for loading modules
527-
*/
528-
@Deprecated
529-
public Authentication getRestAuthentication() {
530-
return restAuthentication;
531-
}
532-
533-
@Deprecated
534-
public void setRestAuthentication(Authentication authentication) {
535-
this.restAuthentication = authentication;
536-
}
537-
538518
/**
539519
* As of 3.3.0, this now returns the first ConfigDir in the List of ConfigsDir that this class now maintains.
540520
*
@@ -862,16 +842,6 @@ public void setAppServicesSslHostnameVerifier(SSLHostnameVerifier appServicesSsl
862842
this.appServicesSslHostnameVerifier = appServicesSslHostnameVerifier;
863843
}
864844

865-
@Deprecated
866-
public Authentication getAppServicesAuthentication() {
867-
return appServicesAuthentication;
868-
}
869-
870-
@Deprecated
871-
public void setAppServicesAuthentication(Authentication appServicesAuthentication) {
872-
this.appServicesAuthentication = appServicesAuthentication;
873-
}
874-
875845
public String getReplicaForestDataDirectory() {
876846
return replicaForestDataDirectory;
877847
}

src/main/java/com/marklogic/appdeployer/DefaultAppConfigFactory.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,6 @@ public AppConfig newAppConfig() {
141141
prop = getProperty("mlRestAuthentication");
142142
if (prop != null) {
143143
logger.info("App REST authentication: " + prop);
144-
c.setRestAuthentication(DatabaseClientFactory.Authentication.valueOfUncased(prop));
145144
c.setRestSecurityContextType(SecurityContextType.valueOf(prop.toUpperCase()));
146145
}
147146

@@ -232,7 +231,6 @@ public AppConfig newAppConfig() {
232231
if (prop != null) {
233232
logger.info("App Services authentication: " + prop);
234233
c.setAppServicesSecurityContextType(SecurityContextType.valueOf(prop.toUpperCase()));
235-
c.setAppServicesAuthentication(DatabaseClientFactory.Authentication.valueOfUncased(prop));
236234
}
237235

238236
prop = getProperty("mlAppServicesCertFile");

src/test/java/com/marklogic/appdeployer/DefaultAppConfigFactoryTest.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,6 @@ public void allProperties() {
203203
assertEquals((Integer) 8765, config.getTestRestPort());
204204
assertEquals("propuser2", config.getRestAdminUsername());
205205
assertEquals("proppassword2", config.getRestAdminPassword());
206-
assertEquals(DatabaseClientFactory.Authentication.CERTIFICATE, config.getRestAuthentication());
207206
assertEquals(SecurityContextType.CERTIFICATE, config.getRestSecurityContextType());
208207
assertEquals("restCertFile", config.getRestCertFile());
209208
assertEquals("restCertPassword", config.getRestCertPassword());
@@ -213,7 +212,6 @@ public void allProperties() {
213212
assertEquals("appServicesUsername", config.getAppServicesUsername());
214213
assertEquals("appServicesPassword", config.getAppServicesPassword());
215214
assertEquals((Integer) 8123, config.getAppServicesPort());
216-
assertEquals(DatabaseClientFactory.Authentication.KERBEROS, config.getAppServicesAuthentication());
217215
assertEquals(SecurityContextType.KERBEROS, config.getAppServicesSecurityContextType());
218216
assertEquals("appServicesCertFile", config.getAppServicesCertFile());
219217
assertEquals("appServicesCertPassword", config.getAppServicesCertPassword());

0 commit comments

Comments
 (0)