diff options
author | Corey Hulen <corey@hulen.com> | 2015-06-15 10:02:11 -0800 |
---|---|---|
committer | Corey Hulen <corey@hulen.com> | 2015-06-15 10:02:11 -0800 |
commit | 916df982523d62dbacb9f0204c6024b0fffc3530 (patch) | |
tree | 79da3340d0c6cf7fa7612fc3cd4ae78b9224e75a | |
parent | 334261ece1bd312678f8eb65b2fde3f636a0475d (diff) | |
parent | adceb8c929b57d8c320b9b39167fb25f04ca99a1 (diff) | |
download | chat-916df982523d62dbacb9f0204c6024b0fffc3530.tar.gz chat-916df982523d62dbacb9f0204c6024b0fffc3530.tar.bz2 chat-916df982523d62dbacb9f0204c6024b0fffc3530.zip |
Merge pull request #4 from mattermost/mm-1245
fixes mm-1245 filenames are now url escaped
-rw-r--r-- | api/file.go | 2 | ||||
-rw-r--r-- | web/react/components/view_image.jsx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/api/file.go b/api/file.go index c7c3b7b3e..10167c6ff 100644 --- a/api/file.go +++ b/api/file.go @@ -114,7 +114,7 @@ func uploadFile(c *Context, w http.ResponseWriter, r *http.Request) { return } - fileUrl := c.TeamUrl + "/api/v1/files/get/" + channelId + "/" + c.Session.UserId + "/" + uid + "/" + files[i].Filename + fileUrl := c.TeamUrl + "/api/v1/files/get/" + channelId + "/" + c.Session.UserId + "/" + uid + "/" + url.QueryEscape(files[i].Filename) resStruct.Filenames = append(resStruct.Filenames, fileUrl) } diff --git a/web/react/components/view_image.jsx b/web/react/components/view_image.jsx index 7d0f0d8a9..4cb30e1d3 100644 --- a/web/react/components/view_image.jsx +++ b/web/react/components/view_image.jsx @@ -165,7 +165,7 @@ module.exports = React.createClass({ <span className="text"> | </span> </div> : "" } - <a href={this.props.filenames[id]} download={name} className="text">Download</a> + <a href={this.props.filenames[id]} download={decodeURIComponent(name)} className="text">Download</a> </div> </div> {loading} |