Remove user data from logs when not in debug/trace mode

Signed-off-by: Mohit Godwani <[email protected]>

server/src/main/java/org/opensearch/action/support/replication/ReplicationOperation.java

@@ -144,10 +144,9 @@ public void execute() throws Exception {
             finishAsFailed(
                 new UnavailableShardsException(
                     primaryId,
-                    "{} Timeout: [{}], request: [{}]",
+                    "{} Timeout: [{}]",
                     activeShardCountFailure,
-                    request.timeout(),
-                    request
+                    request.timeout()
                 )
             );
             return;

server/src/main/java/org/opensearch/action/support/replication/TransportReplicationAction.java

@@ -1246,7 +1246,7 @@ void finishOnSuccess(Response response) {
         }
 
         void retryBecauseUnavailable(ShardId shardId, String message) {
-            retry(new UnavailableShardsException(shardId, "{} Timeout: [{}], request: [{}]", message, request.timeout(), request));
+            retry(new UnavailableShardsException(shardId, "{} Timeout: [{}]", message, request.timeout()));
         }
     }
 

server/src/main/java/org/opensearch/action/update/UpdateHelper.java

@@ -58,6 +58,7 @@
 import org.opensearch.index.shard.IndexShard;
 import org.opensearch.script.Script;
 import org.opensearch.script.ScriptService;
+import org.opensearch.script.ScriptType;
 import org.opensearch.script.UpdateScript;
 import org.opensearch.search.lookup.SourceLookup;
 
@@ -128,7 +129,11 @@ Tuple<UpdateOpType, Map<String, Object>> executeScriptedUpsert(Map<String, Objec
 
         if (operation != UpdateOpType.CREATE && operation != UpdateOpType.NONE) {
             // Only valid options for an upsert script are "create" (the default) or "none", meaning abort upsert
-            logger.warn("Invalid upsert operation [{}] for script [{}], doing nothing...", operation, script.getIdOrCode());
+            if (logger.isDebugEnabled() || ScriptType.STORED.equals(script.getType())) {
+                logger.warn("Invalid upsert operation [{}] for script [{}], doing nothing...", operation, script.getIdOrCode());
+            } else {
+                logger.warn("Invalid upsert operation [{}] for given script", operation);
+            }
             operation = UpdateOpType.NONE;
         }