From d62c3b36242c1efcd2c4db644a30755c06f5ed98 Mon Sep 17 00:00:00 2001
From: Kamalika Saha <kamalikasaha@microsoft.com>
Date: Mon, 4 Mar 2024 18:55:55 +0530
Subject: [PATCH] Removing info log statements from two files as suggested by
community only from master branch.
---
.../provider/azure/subscriber/TopicsRepositoryImpl.java | 4 ----
.../register/subscriber/services/ChallengeResponseCheck.java | 5 -----
2 files changed, 9 deletions(-)
diff --git a/provider/register-azure/src/main/java/org/opengroup/osdu/register/provider/azure/subscriber/TopicsRepositoryImpl.java b/provider/register-azure/src/main/java/org/opengroup/osdu/register/provider/azure/subscriber/TopicsRepositoryImpl.java
index 58f668098..4ca782f98 100644
--- a/provider/register-azure/src/main/java/org/opengroup/osdu/register/provider/azure/subscriber/TopicsRepositoryImpl.java
+++ b/provider/register-azure/src/main/java/org/opengroup/osdu/register/provider/azure/subscriber/TopicsRepositoryImpl.java
@@ -45,7 +45,6 @@ public class TopicsRepositoryImpl implements ITopicsRepository {
public List<Topic> listMessages() {
final String partitionId = headers.getPartitionId();
List<Topic> template = new ArrayList<>();
- List<String> topicNames = new ArrayList<>();
List<Topic> output = new ArrayList<>();
if (!topicsMap.containsKey(partitionId)) {
@@ -55,15 +54,12 @@ public class TopicsRepositoryImpl implements ITopicsRepository {
for (Topic topic: topicsMap.get(partitionId)) {
output.add(topic);
- topicNames.add(topic.getName());
}
if (!featureFlag.isFeatureEnabled(FEATURE_FLAG_NAME)){
output.removeIf(topic -> topic.getName().equals(TOPIC_NAME));
- topicNames.removeIf(topicName -> topicName.equals(TOPIC_NAME));
}
- log.info("List of Topic Names:" + topicNames);
return output;
}
diff --git a/register-core/src/main/java/org/opengroup/osdu/register/subscriber/services/ChallengeResponseCheck.java b/register-core/src/main/java/org/opengroup/osdu/register/subscriber/services/ChallengeResponseCheck.java
index 63522ed07..2d6bb6c1f 100644
--- a/register-core/src/main/java/org/opengroup/osdu/register/subscriber/services/ChallengeResponseCheck.java
+++ b/register-core/src/main/java/org/opengroup/osdu/register/subscriber/services/ChallengeResponseCheck.java
@@ -100,15 +100,11 @@ public class ChallengeResponseCheck {
secretString = this.googleServiceAccount.getPrivateKeyId(keyString);
// send gsa challange with idtoken in header
- log.info("Sending challange request with GSA Secret Type");
response = sendGsaChallenge(input, crc, idToken);
- log.info("Response from GSA challange check endpoint: %s" + response.getResponseCode());
} else {
secretString = ((HmacSecret) secret).getValue();
String hmacSignature = this.signatureService.getSignedSignature(input.getPushEndpoint(), secretString);
- log.info("Sending challange request with HMAC Secret Type");
response = sendHmacChallenge(input, crc, hmacSignature);
- log.info("Response from HMAC challange check endpoint: " + response.getResponseCode());
}
String returnedHash = getChallengeResponseHash(response);
@@ -116,7 +112,6 @@ public class ChallengeResponseCheck {
return encodedHash.equalsIgnoreCase(returnedHash);
} catch (Exception e) {
log.error("Unexpected error during challenge response check.", e);
- log.info(String.format("Exception occurred during Challange Response Check. Exception Details : %s " , e.toString()));
return false;
}
}
--
GitLab