diff options
author | Harrison Healey <harrisonmhealey@gmail.com> | 2018-07-09 13:32:42 -0400 |
---|---|---|
committer | Carlos Tadeu Panato Junior <ctadeu@gmail.com> | 2018-07-09 19:32:42 +0200 |
commit | 28449dd95ae2c550d49c021217518feeb582a5d9 (patch) | |
tree | 8aa1afc4d1078f5ed8a1463c160d5e941d961754 /model | |
parent | 8a02121767e696dac0727a9a064d6d504bc34787 (diff) | |
download | chat-28449dd95ae2c550d49c021217518feeb582a5d9.tar.gz chat-28449dd95ae2c550d49c021217518feeb582a5d9.tar.bz2 chat-28449dd95ae2c550d49c021217518feeb582a5d9.zip |
Update custom url schemes error message to reflect what we actually support (#9061)
Diffstat (limited to 'model')
-rw-r--r-- | model/config.go | 2 | ||||
-rw-r--r-- | model/config_test.go | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/model/config.go b/model/config.go index 7105af893..be940d893 100644 --- a/model/config.go +++ b/model/config.go @@ -2418,7 +2418,7 @@ func (mes *MessageExportSettings) isValid(fs FileSettings) *AppError { func (ds *DisplaySettings) isValid() *AppError { if len(*ds.CustomUrlSchemes) != 0 { - validProtocolPattern := regexp.MustCompile(`(?i)^\s*[a-z][a-z0-9+.-]*\s*$`) + validProtocolPattern := regexp.MustCompile(`(?i)^\s*[a-z][a-z0-9-]*\s*$`) for _, scheme := range *ds.CustomUrlSchemes { if !validProtocolPattern.MatchString(scheme) { diff --git a/model/config_test.go b/model/config_test.go index 5406d680d..848f4327e 100644 --- a/model/config_test.go +++ b/model/config_test.go @@ -466,7 +466,7 @@ func TestDisplaySettingsIsValidCustomUrlSchemes(t *testing.T) { { name: "containing period", value: []string{"iris.beep"}, - valid: true, + valid: false, // should technically be true, but client doesn't support it }, { name: "containing hyphen", @@ -476,7 +476,7 @@ func TestDisplaySettingsIsValidCustomUrlSchemes(t *testing.T) { { name: "containing plus", value: []string{"coap+tcp", "coap+ws"}, - valid: true, + valid: false, // should technically be true, but client doesn't support it }, { name: "starting with number", |