Merge pull request #1087 from PublicisSapient/DTS-37340-codeql-logInj…

…ection DTS-37340 removed logs
PublicisSapient · Jun 12, 2024 · 7e7698d · 7e7698d
2 parents c3a5cb4 + cdddccd
commit 7e7698d
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 12 deletions.
diff --git a/.../main/java/com/publicissapient/kpidashboard/common/controller/RunProcessorController.java b/.../main/java/com/publicissapient/kpidashboard/common/controller/RunProcessorController.java
@@ -60,9 +60,6 @@ public ResponseEntity<Map> runProcessorForProjects(
 		ExecutionLogContext.set(processorExecutionBasicConfig.getLogContext());
 		MDC.put("Processor Name", jobExecuter.getProcessor().getProcessorName());
 		MDC.put("RequestStartTime", String.valueOf(System.currentTimeMillis()));
-		log.info("Received request to run the processor: {} for projects {}",
-				jobExecuter.getProcessor().getProcessorName(),
-				processorExecutionBasicConfig.getProjectBasicConfigIds());
 
 		jobExecuter.setProjectsBasicConfigIds(processorExecutionBasicConfig.getProjectBasicConfigIds());
 		jobExecuter.setExecutionLogContext(ExecutionLogContext.getContext());

diff --git a/...om/publicissapient/kpidashboard/common/service/ProcessorExecutionTraceLogServiceImpl.java b/...om/publicissapient/kpidashboard/common/service/ProcessorExecutionTraceLogServiceImpl.java
@@ -47,12 +47,6 @@ public class ProcessorExecutionTraceLogServiceImpl implements ProcessorExecution
 
 	@Override
 	public void save(ProcessorExecutionTraceLog processorExecutionTracelog) {
-		log.info(
-				"last execution time of {} for project {} is {}. status is {} and lastSuccessfulRun is {} and LastEnableAssigneeToggleState is {} ",
-				processorExecutionTracelog.getProcessorName(), processorExecutionTracelog.getBasicProjectConfigId(),
-				processorExecutionTracelog.getExecutionEndedAt(), processorExecutionTracelog.isExecutionSuccess(),
-				processorExecutionTracelog.getLastSuccessfulRun(),
-				processorExecutionTracelog.isLastEnableAssigneeToggleState());
 
 		Optional<ProcessorExecutionTraceLog> existingTraceLogOptional = processorExecutionTraceLogRepository
 				.findByProcessorNameAndBasicProjectConfigId(processorExecutionTracelog.getProcessorName(),

diff --git a/...licissapient/kpidashboard/apis/projectconfig/basic/rest/ProjectBasicConfigController.java b/...licissapient/kpidashboard/apis/projectconfig/basic/rest/ProjectBasicConfigController.java
@@ -150,8 +150,6 @@ public ResponseEntity<ProjectConfigResponse> addBasicConfig(
 			});
 		}
 
-		log.info(ADDING_PROJECT_CONFIGURATIONS, projectBasicConfigDTO.toString());
-
 		ServiceResponse serviceResp = projectBasicConfigService.addBasicConfig(projectBasicConfigDTO);
 
 		List<RoleWiseProjects> projectAccess = projectAccessManager

diff --git a/customapi/src/main/java/com/publicissapient/kpidashboard/apis/repotools/RepoToolsClient.java b/customapi/src/main/java/com/publicissapient/kpidashboard/apis/repotools/RepoToolsClient.java
@@ -69,7 +69,6 @@ public int enrollProjectCall(RepoToolConfig repoToolConfig, String repoToolsUrl,
 		URI url = URI.create(repoToolsUrl);
 		HttpEntity<String> entity = new HttpEntity<>(payload, httpHeaders);
 		ResponseEntity<String> response = restTemplate.exchange(url, HttpMethod.POST, entity, String.class);
-		log.debug(response.getBody());
 		return response.getStatusCode().value();
 	}