Make OTP response optional and toggle via config (#657)

AmirRajabii · web-flow · commit 7233c4830343 · 2026-04-08T14:01:36.000+03:30
diff --git a/api/api-core/src/main/kotlin/co/nilin/opex/api/core/inout/TempOtpResponse.kt b/api/api-core/src/main/kotlin/co/nilin/opex/api/core/inout/TempOtpResponse.kt
@@ -1,3 +1,3 @@
 package co.nilin.opex.api.core.inout
 
-data class TempOtpResponse(val otp: String, var receivers: List<OTPReceiver>? = null)
+data class TempOtpResponse(val otp: String?, var receivers: List<OTPReceiver>? = null)
diff --git a/auth-gateway/auth-gateway-app/src/main/kotlin/co/nilin/opex/auth/model/OTP.kt b/auth-gateway/auth-gateway-app/src/main/kotlin/co/nilin/opex/auth/model/OTP.kt
@@ -26,7 +26,7 @@ data class OTPVerifyResponse(
 )
 
 //TODO IMPORTANT: remove in production
-data class TempOtpResponse(val otp: String, val otpReceiver: OTPReceiver?)
+data class TempOtpResponse(val otp: String?, val otpReceiver: OTPReceiver?)
 
 enum class OTPAction {
     REGISTER, FORGET, NONE
diff --git a/docker-compose.yml b/docker-compose.yml
@@ -545,6 +545,7 @@ services:
       - SMTP_PASS=${SMTP_PASS}
       - SMTP_FROM=${SMTP_FROM}
       - TOKEN_ISSUER_URL=${KC_ISSUER_URL}
+      - OTP_CODE_RESPONSE_ENABLED=${OTP_CODE_RESPONSE_ENABLED}
     depends_on:
       - consul
       - postgres-otp
diff --git a/otp/otp-app/src/main/kotlin/co/nilin/opex/otp/app/controller/OTPController.kt b/otp/otp-app/src/main/kotlin/co/nilin/opex/otp/app/controller/OTPController.kt
@@ -6,6 +6,7 @@ import co.nilin.opex.otp.app.data.OTPResult
 import co.nilin.opex.otp.app.data.VerifyOTPRequest
 import co.nilin.opex.otp.app.model.OTPType
 import co.nilin.opex.otp.app.service.OTPService
+import org.springframework.beans.factory.annotation.Value
 import org.springframework.http.HttpStatus
 import org.springframework.http.ResponseEntity
 import org.springframework.web.bind.annotation.PostMapping
@@ -15,10 +16,13 @@ import org.springframework.web.bind.annotation.RestController
 
 @RestController
 @RequestMapping("/v1/otp")
-class OTPController(private val otpService: OTPService) {
+class OTPController(
+    private val otpService: OTPService,
+    @Value("\${otp.response-enabled}") private val otpCodeResponseEnabled: Boolean,
+) {
 
     //TODO IMPORTANT: remove in production
-    data class TempOtpResponse(val otp: String)
+    data class TempOtpResponse(val otp: String?)
     //TODO IMPORTANT: remove in production
 
     //TODO IMPORTANT: remove in production
@@ -34,7 +38,8 @@ class OTPController(private val otpService: OTPService) {
             )
         else
             otpService.requestCompositeOTP(request.receivers.toSet(), request.userId, request.action)
-        return ResponseEntity.status(HttpStatus.CREATED).body(TempOtpResponse(code))
+        val tempOtpResponse = if (otpCodeResponseEnabled) code else null
+        return ResponseEntity.status(HttpStatus.CREATED).body(TempOtpResponse(tempOtpResponse))
     }
 
     @PostMapping("/verify")
diff --git a/otp/otp-app/src/main/resources/application.yml b/otp/otp-app/src/main/resources/application.yml
@@ -48,6 +48,7 @@ app:
     cert-url: http://keycloak:8080/realms/opex/protocol/openid-connect/certs
     iss-url: ${TOKEN_ISSUER_URL:http://keycloak:8080/realms/opex}
 otp:
+  response-enabled: ${OTP_CODE_RESPONSE_ENABLED:false}
   sms:
     provider:
       url: ${SMS_PROVIDER_URL}
diff --git a/profile/profile-core/src/main/kotlin/co/nilin/opex/profile/core/data/otp/TempOtpResponse.kt b/profile/profile-core/src/main/kotlin/co/nilin/opex/profile/core/data/otp/TempOtpResponse.kt
@@ -1,3 +1,3 @@
 package co.nilin.opex.profile.core.data.otp
 
-data class TempOtpResponse(val otp: String, var otpReceiver: OTPReceiver?)
+data class TempOtpResponse(val otp: String?, var otpReceiver: OTPReceiver?)
diff --git a/wallet/wallet-app/src/main/kotlin/co/nilin/opex/wallet/app/service/WalletSnapshotService.kt b/wallet/wallet-app/src/main/kotlin/co/nilin/opex/wallet/app/service/WalletSnapshotService.kt
@@ -29,7 +29,7 @@ class WalletSnapshotService(
         }
     }
 
-    @Scheduled(cron = "0 40 16 * * ?", zone = "GMT" + "\${app.zone-offset}")
+    @Scheduled(cron = "0 0 0 * * ?", zone = "GMT" + "\${app.zone-offset}")
     fun createDetailAssetsSnapshot() {
         runBlocking {
             updatePrices()
diff --git a/wallet/wallet-core/src/main/kotlin/co/nilin/opex/wallet/core/inout/otp/TempOtpResponse.kt b/wallet/wallet-core/src/main/kotlin/co/nilin/opex/wallet/core/inout/otp/TempOtpResponse.kt
@@ -1,3 +1,3 @@
 package co.nilin.opex.wallet.core.inout.otp
 
-data class TempOtpResponse(val otp: String, var receivers: List<OTPReceiver>? = null)
+data class TempOtpResponse(val otp: String?, var receivers: List<OTPReceiver>? = null)

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`package co.nilin.opex.api.core.inout`
`2`	`2`
`3`		`-data class TempOtpResponse(val otp: String, var receivers: List<OTPReceiver>? = null)`
	`3`	`+data class TempOtpResponse(val otp: String?, var receivers: List<OTPReceiver>? = null)`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ data class OTPVerifyResponse(`
`26`	`26`	`)`
`27`	`27`
`28`	`28`	`//TODO IMPORTANT: remove in production`
`29`		`-data class TempOtpResponse(val otp: String, val otpReceiver: OTPReceiver?)`
	`29`	`+data class TempOtpResponse(val otp: String?, val otpReceiver: OTPReceiver?)`
`30`	`30`
`31`	`31`	`enum class OTPAction {`
`32`	`32`	`REGISTER, FORGET, NONE`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`package co.nilin.opex.profile.core.data.otp`
`2`	`2`
`3`		`-data class TempOtpResponse(val otp: String, var otpReceiver: OTPReceiver?)`
	`3`	`+data class TempOtpResponse(val otp: String?, var otpReceiver: OTPReceiver?)`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ class WalletSnapshotService(`
`29`	`29`	`}`
`30`	`30`	`}`
`31`	`31`
`32`		`- @Scheduled(cron = "0 40 16 * * ?", zone = "GMT" + "\${app.zone-offset}")`
	`32`	`+ @Scheduled(cron = "0 0 0 * * ?", zone = "GMT" + "\${app.zone-offset}")`
`33`	`33`	`fun createDetailAssetsSnapshot() {`
`34`	`34`	`runBlocking {`
`35`	`35`	`updatePrices()`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`package co.nilin.opex.wallet.core.inout.otp`
`2`	`2`
`3`		`-data class TempOtpResponse(val otp: String, var receivers: List<OTPReceiver>? = null)`
	`3`	`+data class TempOtpResponse(val otp: String?, var receivers: List<OTPReceiver>? = null)`