From 96ff3cd05a2bd09c8ac607615234de9b5b957b00 Mon Sep 17 00:00:00 2001 From: Joachim Hill-Grannec Date: Fri, 12 Aug 2016 14:20:26 -0700 Subject: [PATCH] Makes more sense for the author value to only have the name since other plugins may use this value. --- remote/bitbucketserver/convert.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/remote/bitbucketserver/convert.go b/remote/bitbucketserver/convert.go index ac09ce537..bbe2c046d 100644 --- a/remote/bitbucketserver/convert.go +++ b/remote/bitbucketserver/convert.go @@ -69,9 +69,9 @@ func convertPushHook(hook *internal.PostHook, baseURL string) *model.Build { //Ensuring the author label is not longer then 40 for the label of the commit author (default size in the db) - authorLabel := fmt.Sprintf("%s <%s>", hook.Changesets.Values[0].ToCommit.Author.Name, hook.Changesets.Values[0].ToCommit.Author.EmailAddress) + authorLabel := hook.Changesets.Values[0].ToCommit.Author.Name if (len(authorLabel) > 40) { - authorLabel = fmt.Sprintf("%s", authorLabel[0:40]) + authorLabel = authorLabel[0:40] } build := &model.Build{