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

Commit 4a5446d

Browse files
committed
#381 Softened logging language for what roles are needed
For some security resources like roles and privileges, new privileges can be substituted for the security role, but that's a bit too difficult to explain in a logging message
1 parent c28fa20 commit 4a5446d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/main/java/com/marklogic/mgmt/ManageClient.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public void setManageConfig(ManageConfig config) {
6969
String securityUsername = config.getSecurityUsername();
7070
if (securityUsername != null && securityUsername.trim().length() > 0 && !securityUsername.equals(config.getUsername())) {
7171
if (logger.isInfoEnabled()) {
72-
logger.info(format("Initializing separate connection to Manage API with user '%s' that must have both manage-admin and security roles", securityUsername));
72+
logger.info(format("Initializing separate connection to Manage API with user '%s' that should have the 'manage-admin' and 'security' roles", securityUsername));
7373
}
7474

7575
RestConfig rc = new RestConfig(config.getHost(), config.getPort(), securityUsername, config.getSecurityPassword());
@@ -368,7 +368,7 @@ protected void logRequest(String path, String contentType, String method) {
368368

369369
protected void logSecurityUserRequest(String path, String contentType, String method) {
370370
if (logger.isInfoEnabled()) {
371-
logger.info(String.format("Sending %s %s request as user '%s' (who must have the 'manage-admin' and 'security' roles) to path: %s",
371+
logger.info(String.format("Sending %s %s request as user '%s' (who should have the 'manage-admin' and 'security' roles) to path: %s",
372372
contentType, method, determineUsernameForSecurityUserRequest(), path));
373373
}
374374
}

0 commit comments

Comments
 (0)