Avoid nil safe message (!21326) · Merge requests · GitLab.org / GitLab FOSS · GitLab

Sean McGivern requested to merge yisiliang/gitlab-ce-avoid-nil-safe_message into master Aug 21, 2018

Continuation of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16889 - thanks @yisiliang!

What does this MR do?

This avoids a 500 error reported in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16889. We already handle safe_commit_message being blank when getting the title, but we didn't with the description.

Does this MR meet the acceptance criteria?

Changelog entry added, if necessary
Tests added for this feature/bug
Conforms to the code review guidelines
Conforms to the merge request performance guidelines
Conforms to the style guides

Edited Aug 21, 2018 by Sean McGivern