From 4781a8dc5e3074ff9d2eb17f0a902b612f213cb5 Mon Sep 17 00:00:00 2001
From: demirkayaender <80807029+demirkayaender@users.noreply.github.com>
Date: Tue, 17 Aug 2021 17:30:37 -0700
Subject: [PATCH] Expose invalid timer value in the error message (#4380)

---
 service/history/decision/checker.go | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/service/history/decision/checker.go b/service/history/decision/checker.go
index 9fa11d91a4e..db2886aefba 100644
--- a/service/history/decision/checker.go
+++ b/service/history/decision/checker.go
@@ -363,7 +363,9 @@ func (v *attrValidator) validateTimerScheduleAttributes(
 		return &types.BadRequestError{Message: "TimerId exceeds length limit."}
 	}
 	if attributes.GetStartToFireTimeoutSeconds() <= 0 {
-		return &types.BadRequestError{Message: "A valid StartToFireTimeoutSeconds is not set on decision."}
+		return &types.BadRequestError{
+			Message: fmt.Sprintf("Invalid StartToFireTimeoutSeconds: %v", attributes.GetStartToFireTimeoutSeconds()),
+		}
 	}
 	return nil
 }