Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

[MM-13147] Fix for file_upload crash when mimeTypesSuffix is not present #2074

Merged
merged 6 commits into from
Nov 23, 2018

Conversation

sudheerDev
Copy link
Contributor

@sudheerDev sudheerDev commented Nov 23, 2018

Summary

Fix for file_upload crash for certain mimeTypes

Ticket Link

MM-13147

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Ran make check-style to check for style errors (required for all pull requests)
  • Ran make test to ensure unit and component tests passed
  • Added or updated unit tests (required for all new features)

@sudheerDev sudheerDev added the 2: Dev Review Requires review by a core commiter label Nov 23, 2018
@sudheerDev sudheerDev added this to the v5.6.0 milestone Nov 23, 2018
@enahum enahum changed the title [MM-13147] Fix for file_upload crash for certain mimeTypes [MM-13147] Fix for file_upload crash when mimeTypesSuffix is not present Nov 23, 2018
@sudheerDev sudheerDev added the Setup Old Test Server Triggers the creation of a test server label Nov 23, 2018

it('mime type for no suffix', () => {
assert.equal(getFileTypeFromMime('asdasd'), 'other');
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@saturninoabril saturninoabril added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Nov 23, 2018
@enahum enahum closed this Nov 23, 2018
@enahum enahum reopened this Nov 23, 2018
@enahum enahum removed 4: Reviews Complete All reviewers have approved the pull request Setup Old Test Server Triggers the creation of a test server labels Nov 23, 2018
@mattermost mattermost deleted a comment from mattermod Nov 23, 2018
@enahum enahum closed this Nov 23, 2018
@enahum enahum reopened this Nov 23, 2018
@mattermod
Copy link
Contributor

Spintmint test server creation failed. See the logs for more information.

@enahum
Copy link
Contributor

enahum commented Nov 23, 2018

@sudheerDev build failed

utils/file_utils.jsx Outdated Show resolved Hide resolved
utils/file_utils.jsx Outdated Show resolved Hide resolved
utils/file_utils.jsx Outdated Show resolved Hide resolved
utils/file_utils.jsx Outdated Show resolved Hide resolved
utils/file_utils.jsx Outdated Show resolved Hide resolved
@enahum enahum added the 2: Dev Review Requires review by a core commiter label Nov 23, 2018
@mattermod
Copy link
Contributor

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

@enahum enahum added the Setup Old Test Server Triggers the creation of a test server label Nov 23, 2018
@mattermod
Copy link
Contributor

Spinmint test server created at: https://i-077f837a25a01520b.test.spinmint.com

Test Admin Account: Username: sysadmin | Password: sysadmin

Test User Account: Username: user-1 | Password: user-1

Instance ID: i-077f837a25a01520b

@enahum enahum removed the Setup Old Test Server Triggers the creation of a test server label Nov 23, 2018
@mattermod
Copy link
Contributor

Spinmint test server destroyed

@enahum enahum added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Nov 23, 2018
@enahum enahum merged commit a31d34d into mattermost:master Nov 23, 2018
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Nov 23, 2018
@lindy65 lindy65 added Tests/Not Needed Does not require new release tests and removed 4: Reviews Complete All reviewers have approved the pull request labels Dec 4, 2018
@JtheBAB JtheBAB mentioned this pull request Jan 15, 2019
9 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
6 participants