fix(telemetry): sns subject length (#1463)

break before reaching 100 characters limit as limit is `<100`
PeerDB-io · Mar 11, 2024 · f7a823f · f7a823f
1 parent 7d6c489
commit f7a823f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/flow/shared/telemetry/sns_message_sender.go b/flow/shared/telemetry/sns_message_sender.go
@@ -42,14 +42,14 @@ func (s *SNSMessageSenderImpl) SendMessage(ctx context.Context, subject string,
 	var messageSubjectBuilder strings.Builder
 	maxSubjectSize := 99
 	for currentLength, char := range subject {
+		if currentLength > maxSubjectSize {
+			break
+		}
 		if unicode.IsPrint(char) {
 			messageSubjectBuilder.WriteRune(char)
 		} else {
 			messageSubjectBuilder.WriteRune(' ')
 		}
-		if currentLength > maxSubjectSize {
-			break
-		}
 	}
 	publish, err := s.client.Publish(ctx, &sns.PublishInput{
 		Message: aws.String(body),