Skip to content

Commit f0e2bb5

Browse files
committed
notify: verify passowrd isn't leaked from Email sender
1 parent 3e2cb28 commit f0e2bb5

File tree

1 file changed

+20
-0
lines changed

1 file changed

+20
-0
lines changed

internal/notify/multi_test.go

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,26 @@ func TestMultiSender(t *testing.T) {
3636
require.NoError(t, sender.Critical(ctx, msg))
3737
}
3838

39+
func TestMultiSender_senderTypes(t *testing.T) {
40+
logger := log.NewTestLogger()
41+
cfg := &service.Notifications{
42+
Email: []service.Email{
43+
{
44+
ID: "testing",
45+
From: "user:pass@localhost:4133",
46+
},
47+
},
48+
}
49+
notifiers := &service.UploadNotifiers{
50+
Email: []string{"testing"},
51+
}
52+
53+
sender, err := NewMultiSender(logger, cfg, notifiers)
54+
require.NoError(t, err)
55+
56+
require.Equal(t, "*notify.Email", sender.senderTypes()) // no password leaked
57+
}
58+
3959
func TestMultiSenderErr(t *testing.T) {
4060
sendErr := errors.New("bad error")
4161

0 commit comments

Comments
 (0)