Update api/pkg/validators/bulk_message_handler_validator.go

AchoArnold · Copilot · web-flow · commit c1dddc36c944 · 2026-03-22T14:35:00.000+02:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/api/pkg/validators/bulk_message_handler_validator.go b/api/pkg/validators/bulk_message_handler_validator.go
@@ -241,13 +241,10 @@ func (v *BulkMessageHandlerValidator) validateMessages(ctx context.Context, mess
 					result.Add("document", fmt.Sprintf("Row [%d]: The attachment URL [%s] has an invalid url format.", index+2, cleanURL))
 				} else if parsedURL.Scheme != "http" && parsedURL.Scheme != "https" {
 					result.Add("document", fmt.Sprintf("Row [%d]: The attachment URL [%s] must use http or https.", index+2, cleanURL))
-				} else {
-					if err := validateAttachmentURL(ctx, v.cache, cleanURL); err != nil {
-						result.Add("attachments", fmt.Sprintf("Row [%d]: The attachment URL [%s] failed validation: %s", index+2, cleanURL, err.Error()))
-					}
 				}
 			}
 		}
+		}
 
 		if _, err := phonenumbers.Parse(message.FromPhoneNumber, phonenumbers.UNKNOWN_REGION); err != nil {
 			result.Add("document", fmt.Sprintf("Row [%d]: The FromPhoneNumber [%s] is not a valid E.164 phone number", index+2, message.FromPhoneNumber))

Original file line number	Diff line number	Diff line change
`@@ -241,13 +241,10 @@ func (v *BulkMessageHandlerValidator) validateMessages(ctx context.Context, mess`
`241`	`241`	`result.Add("document", fmt.Sprintf("Row [%d]: The attachment URL [%s] has an invalid url format.", index+2, cleanURL))`
`242`	`242`	`} else if parsedURL.Scheme != "http" && parsedURL.Scheme != "https" {`
`243`	`243`	`result.Add("document", fmt.Sprintf("Row [%d]: The attachment URL [%s] must use http or https.", index+2, cleanURL))`
`244`		`- } else {`
`245`		`- if err := validateAttachmentURL(ctx, v.cache, cleanURL); err != nil {`
`246`		`- result.Add("attachments", fmt.Sprintf("Row [%d]: The attachment URL [%s] failed validation: %s", index+2, cleanURL, err.Error()))`
`247`		`- }`
`248`	`244`	`}`
`249`	`245`	`}`
`250`	`246`	`}`
	`247`	`+ }`
`251`	`248`
`252`	`249`	`if _, err := phonenumbers.Parse(message.FromPhoneNumber, phonenumbers.UNKNOWN_REGION); err != nil {`
`253`	`250`	`result.Add("document", fmt.Sprintf("Row [%d]: The FromPhoneNumber [%s] is not a valid E.164 phone number", index+2, message.FromPhoneNumber))`