IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix image url replacement for blank urls (#8184)

Browse Source
This commit is contained in:
Chris
2018-02-01 20:31:49 -06:00
committed by Christopher Speller
parent 5d7ae48841
commit 0cd2895456
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/post.go
View File

@@ -890,7 +890,7 @@ func (a *App) ImageProxyAdder() func(string) string {
}
return func(url string) string {
if strings.HasPrefix(url, proxyURL) {
if url == "" || strings.HasPrefix(url, proxyURL) {
return url
}

6
app/post_test.go
View File

@@ -211,6 +211,12 @@ func TestImageProxy(t *testing.T) {
ImageURL: "http://mydomain.com/myimage",
ProxiedImageURL: "https://127.0.0.1/x1000/http://mydomain.com/myimage",
},
"willnorris/imageproxy_EmptyImageURL": {
ProxyType: "willnorris/imageproxy",
ProxyURL: "https://127.0.0.1",
ImageURL: "",
ProxiedImageURL: "",
},
"willnorris/imageproxy_WithSigning": {
ProxyType: "willnorris/imageproxy",
ProxyURL: "https://127.0.0.1",