Webhook Sending Functions Seem to Use the Regular JSON Encoder Which Escapes HTML Characters Unnecessarily about slack

[zoominGiladSher]

Extra context: I saw this in

slack/webhooks.go

Lines 41 to 63 in e715221

	func PostWebhookCustomHTTPContext(ctx context.Context, url string, httpClient *http.Client, msg *WebhookMessage) error {
	raw, err := json.Marshal(msg)
	if err != nil {
	return fmt.Errorf("marshal failed: %w", err)
	}
	req, err := http.NewRequestWithContext(ctx, http.MethodPost, url, bytes.NewReader(raw))
	if err != nil {
	return fmt.Errorf("failed new request: %w", err)
	}
	req.Header.Set("Content-Type", "application/json")
	resp, err := httpClient.Do(req)
	if err != nil {
	return fmt.Errorf("failed to post webhook: %w", err)
	}
	defer func() {
	io.Copy(ioutil.Discard, resp.Body)
	resp.Body.Close()
	}()
	return checkStatusCode(resp, discard{})
	}

and see that there's no use of the EscapeMessage function mentioned in

slack/slackutilsx/slackutilsx.go

Lines 53 to 59 in e715221

	// initialize replacer only once (if needed)
	var escapeReplacer = strings.NewReplacer("&", "&amp;", "<", "&lt;", ">", "&gt;")
	// EscapeMessage text
	func EscapeMessage(message string) string {
	return escapeReplacer.Replace(message)
	}

Not sure if this is the solution, but this is what I saw from a very shallow look at things. Please let me know if I can help in any way or if you'd like me to try and tackle this myself!