RepoMirrors/alertmanager
1
0
Fork 0
mirror of https://github.com/prometheus/alertmanager synced 2025-02-16 10:37:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

notify/webhook: trim space from url_file content

Browse Source 
Signed-off-by: Davide Bizzarri <davide.bizzarri@neoom.com>
This commit is contained in:
Davide Bizzarri 2023-05-16 13:27:22 +02:00 committed by gotjosh
parent 80cbee0f58
commit 6b3dbc1e38
No known key found for this signature in database
GPG Key ID: A6E1DDE38FF3C74E
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
notify/webhook/webhook.go
View File

@ -21,6 +21,7 @@ import (
"io"
"net/http"
"os"
"strings"
"github.com/go-kit/log"
"github.com/go-kit/log/level"
@ -110,7 +111,7 @@ func (n *Notifier) Notify(ctx context.Context, alerts ...*types.Alert) (bool, er
if err != nil {
return false, fmt.Errorf("read url_file: %w", err)
}
url = string(content)
url = strings.TrimSpace(string(content))
}
resp, err := notify.PostJSON(ctx, n.client, url, &buf)

2
notify/webhook/webhook_test.go
View File

@ -124,7 +124,7 @@ func TestWebhookReadingURLFromFile(t *testing.T) {
f, err := os.CreateTemp("", "webhook_url")
require.NoError(t, err, "creating temp file failed")
_, err = f.WriteString(u.String())
_, err = f.WriteString(u.String() + "\n")
require.NoError(t, err, "writing to temp file failed")
notifier, err := New(