mirror of
https://github.com/0rangebananaspy/authelia.git
synced 2024-09-14 22:47:21 +07:00
ab8f9b0697
This change addresses an issue with the usage of the full sender configuration option in the MAIL FROM SMTP command. If a user includes a name in the sender this shouldn't be sent in the MAIL FROM command, instead we should extract it and use just the email portion. Fixes #2571
50 lines
1.3 KiB
Go
50 lines
1.3 KiB
Go
package notification
|
|
|
|
import (
|
|
"crypto/tls"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
|
|
"github.com/authelia/authelia/v4/internal/configuration/schema"
|
|
)
|
|
|
|
func TestShouldConfigureSMTPNotifierWithTLS11(t *testing.T) {
|
|
config := &schema.NotifierConfiguration{
|
|
DisableStartupCheck: true,
|
|
SMTP: &schema.SMTPNotifierConfiguration{
|
|
Host: "smtp.example.com",
|
|
Port: 25,
|
|
TLS: &schema.TLSConfig{
|
|
ServerName: "smtp.example.com",
|
|
MinimumVersion: "TLS1.1",
|
|
},
|
|
},
|
|
}
|
|
|
|
notifier := NewSMTPNotifier(config.SMTP, nil)
|
|
|
|
assert.Equal(t, "smtp.example.com", notifier.tlsConfig.ServerName)
|
|
assert.Equal(t, uint16(tls.VersionTLS11), notifier.tlsConfig.MinVersion)
|
|
assert.False(t, notifier.tlsConfig.InsecureSkipVerify)
|
|
}
|
|
|
|
func TestShouldConfigureSMTPNotifierWithServerNameOverrideAndDefaultTLS12(t *testing.T) {
|
|
config := &schema.NotifierConfiguration{
|
|
DisableStartupCheck: true,
|
|
SMTP: &schema.SMTPNotifierConfiguration{
|
|
Host: "smtp.example.com",
|
|
Port: 25,
|
|
TLS: &schema.TLSConfig{
|
|
ServerName: "smtp.golang.org",
|
|
},
|
|
},
|
|
}
|
|
|
|
notifier := NewSMTPNotifier(config.SMTP, nil)
|
|
|
|
assert.Equal(t, "smtp.golang.org", notifier.tlsConfig.ServerName)
|
|
assert.Equal(t, uint16(tls.VersionTLS12), notifier.tlsConfig.MinVersion)
|
|
assert.False(t, notifier.tlsConfig.InsecureSkipVerify)
|
|
}
|