diff options
| author | Andrew Newdigate <andrew@gitlab.com> | 2017-10-05 22:23:02 +0300 |
|---|---|---|
| committer | Andrew Newdigate <andrew@gitlab.com> | 2017-10-05 22:23:02 +0300 |
| commit | ceee32e2c9de5add07dfd3dd8b3ad80321f397b5 (patch) | |
| tree | 7bc3d8e0f36b21f29fc5a520eed508f2f3abf228 | |
| parent | f19a884487b900fb73b8c413c9779feb97c78841 (diff) | |
| parent | 40f0a1b11ac86e4bba654a9a42a878d2c28f3c1d (diff) | |
Merge branch '615-fix-subject-parsing' into 'master'
Fix Commit Subject parsing in rubyserver
Closes #615
See merge request gitlab-org/gitaly!388
| -rw-r--r-- | ruby/lib/gitaly_server/utils.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ruby/lib/gitaly_server/utils.rb b/ruby/lib/gitaly_server/utils.rb index 08047cced..2cbe57e70 100644 --- a/ruby/lib/gitaly_server/utils.rb +++ b/ruby/lib/gitaly_server/utils.rb @@ -1,9 +1,10 @@ module GitalyServer module Utils def gitaly_commit_from_rugged(rugged_commit) + message_split = rugged_commit.message.split("\n", 2) Gitaly::GitCommit.new( id: rugged_commit.oid, - subject: rugged_commit.message.split("\n", 2)[0].chomp.b, + subject: message_split[0] ? message_split[0].chomp.b : "", body: rugged_commit.message.b, parent_ids: rugged_commit.parent_ids, author: gitaly_commit_author_from_rugged(rugged_commit.author), |