Merge branch 'main' into better-sf-wg-error

PeerDB-io · Mar 11, 2024 · 5d81dcf · 5d81dcf
2 parents 9bae068 + f7a823f
commit 5d81dcf
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),