Minor fix to OTP generation time setting

Signed-off-by: Loganathan Sekar <[email protected]>
mosip · Feb 9, 2024 · 89f76bb · 89f76bb
1 parent 5ea55ac
commit 89f76bb
Showing 1 changed file with 12 additions and 20 deletions.
diff --git a/...n-common/src/main/java/io/mosip/authentication/common/service/integration/OTPManager.java b/...n-common/src/main/java/io/mosip/authentication/common/service/integration/OTPManager.java
@@ -118,26 +118,18 @@ public boolean sendOtp(OtpRequestDTO otpRequestDTO, String idvid, String idvidTy
 				+ EnvUtil.getKeySplitter() + otpRequestDTO.getTransactionID()
 				+ EnvUtil.getKeySplitter() + otp).getBytes());
 
-		OtpTransaction otpTxn;
-		if (otpEntityOpt.isPresent()
-				&& (otpTxn = otpEntityOpt.get()).getStatusCode().equals(IdAuthCommonConstants.ACTIVE_STATUS)) {
-			otpTxn.setOtpHash(otpHash);
-			otpTxn.setUpdBy(securityManager.getUser());
-			otpTxn.setUpdDTimes(otpGenerationTime);
-			otpTxn.setExpiryDtimes(otpGenerationTime.plusSeconds(EnvUtil.getOtpExpiryTime()));
-			otpRepo.save(otpTxn);
-		} else {
-			OtpTransaction txn = new OtpTransaction();
-			txn.setId(UUID.randomUUID().toString());
-			txn.setRefId(securityManager.hash(otpRequestDTO.getIndividualId()));
-			txn.setOtpHash(otpHash);
-			txn.setCrBy(securityManager.getUser());
-			txn.setCrDtimes(otpGenerationTime);
-			txn.setExpiryDtimes(otpGenerationTime.plusSeconds(
-					EnvUtil.getOtpExpiryTime()));
-			txn.setStatusCode(IdAuthCommonConstants.ACTIVE_STATUS);
-			otpRepo.save(txn);
-		}
+		OtpTransaction txn = new OtpTransaction();
+		txn.setId(UUID.randomUUID().toString());
+		txn.setRefId(securityManager.hash(otpRequestDTO.getIndividualId()));
+		txn.setOtpHash(otpHash);
+		txn.setCrBy(securityManager.getUser());
+		txn.setGeneratedDtimes(otpGenerationTime);
+		txn.setCrDtimes(otpGenerationTime);
+		txn.setExpiryDtimes(otpGenerationTime.plusSeconds(
+				EnvUtil.getOtpExpiryTime()));
+		txn.setStatusCode(IdAuthCommonConstants.ACTIVE_STATUS);
+		otpRepo.save(txn);
+
 		String notificationProperty = null;
 		notificationProperty = otpRequestDTO
 				.getOtpChannel().stream().map(channel -> NotificationType.getNotificationTypeForChannel(channel)