From c558deef274d838aae5c0366ff7dc2ebce27a981 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Thu, 20 Sep 2012 09:41:28 -0400 Subject: [PATCH] Support X-Forwarded-Context for subdomain proxy configs (issue 135) --- groovy/sendmail.groovy | 48 ++++++++++++++++++++++++------------------------ 1 files changed, 24 insertions(+), 24 deletions(-) diff --git a/groovy/sendmail.groovy b/groovy/sendmail.groovy index 9616909..c832bc6 100644 --- a/groovy/sendmail.groovy +++ b/groovy/sendmail.groovy @@ -52,12 +52,17 @@ * chain, "return false" at the appropriate failure points. * * Bound Variables: - * gitblit Gitblit Server com.gitblit.GitBlit - * repository Gitblit Repository com.gitblit.models.RepositoryModel - * user Gitblit User com.gitblit.models.UserModel - * commands JGit commands Collection<org.eclipse.jgit.transport.ReceiveCommand> - * url Base url for Gitblit String - * logger Logger instance org.slf4j.Logger + * gitblit Gitblit Server com.gitblit.GitBlit + * repository Gitblit Repository com.gitblit.models.RepositoryModel + * receivePack JGit Receive Pack org.eclipse.jgit.transport.ReceivePack + * user Gitblit User com.gitblit.models.UserModel + * commands JGit commands Collection<org.eclipse.jgit.transport.ReceiveCommand> + * url Base url for Gitblit String + * logger Logs messages to Gitblit org.slf4j.Logger + * clientLogger Logs messages to Git client com.gitblit.utils.ClientLogger + * + * Accessing Gitblit Custom Fields: + * def myCustomField = repository.customFields.myCustomField * */ @@ -102,10 +107,11 @@ toAddresses.addAll(repository.mailingLists) // define the summary and commit urls -def repo = repository.name.replace('/', gitblit.getString(Keys.web.forwardSlashCharacter, '/')) +def repo = repository.name def summaryUrl def commitUrl -if (gitblit.getBoolean(Keys.web.mountParameters, true)) { +if (gitblit.getBoolean(Keys.web.mountParameters, true)) { + repo = repo.replace('/', gitblit.getString(Keys.web.forwardSlashCharacter, '/')).replace('/', '%2F') summaryUrl = url + "/summary/$repo" commitUrl = url + "/commit/$repo/" } else { @@ -122,34 +128,28 @@ def table = { "\n ${JGitUtils.getDisplayName(it.authorIdent)}\n ${df.format(JGitUtils.getCommitDate(it))}\n\n $it.shortMessage\n\n $commitUrl$it.id.name" } for (command in commands) { def ref = command.refName + def refType = 'branch' if (ref.startsWith('refs/heads/')) { ref = command.refName.substring('refs/heads/'.length()) } else if (ref.startsWith('refs/tags/')) { ref = command.refName.substring('refs/tags/'.length()) + refType = 'tag' } switch (command.type) { case ReceiveCommand.Type.CREATE: def commits = JGitUtils.getRevLog(r, command.oldId.name, command.newId.name).reverse() commitCount += commits.size() - if (commits.size() > 0) { - // new branch - changes += "$branchBreak new branch $ref created ($commits.size commits)\n$branchBreak" - changes += commits.collect(table).join(commitBreak) - changes += '\n' - } else if (ref.command.refName.startsWith('refs/tags/')) { - // new tag - changes += "$branchBreak new tag $ref created\n$branchBreak" - } else if (ref.command.refName.startsWith('refs/heads/')) { - // new branch - changes += "$branchBreak new $ref branch created\n$branchBreak" - } + // new branch + changes += "\n$branchBreak new $refType $ref created ($commits.size commits)\n$branchBreak" + changes += commits.collect(table).join(commitBreak) + changes += '\n' break case ReceiveCommand.Type.UPDATE: def commits = JGitUtils.getRevLog(r, command.oldId.name, command.newId.name).reverse() commitCount += commits.size() // fast-forward branch commits table - changes += "$branchBreak $ref branch updated ($commits.size commits)\n$branchBreak" + changes += "\n$branchBreak $ref $refType updated ($commits.size commits)\n$branchBreak" changes += commits.collect(table).join(commitBreak) changes += '\n' break @@ -157,13 +157,13 @@ def commits = JGitUtils.getRevLog(r, command.oldId.name, command.newId.name).reverse() commitCount += commits.size() // non-fast-forward branch commits table - changes += "$branchBreak $ref branch updated [NON fast-forward] ($commits.size commits)\n$branchBreak" + changes += "\n$branchBreak $ref $refType updated [NON fast-forward] ($commits.size commits)\n$branchBreak" changes += commits.collect(table).join(commitBreak) changes += '\n' break case ReceiveCommand.Type.DELETE: // deleted branch/tag - changes += "$branchBreak $ref deleted\n$branchBreak" + changes += "\n$branchBreak $ref $refType deleted\n$branchBreak" break default: break @@ -173,4 +173,4 @@ r.close() // tell Gitblit to send the message (Gitblit filters duplicate addresses) -gitblit.sendMail("$emailprefix $user.username pushed $commitCount commits => $repository.name", "$summaryUrl\n\n$changes", toAddresses) \ No newline at end of file +gitblit.sendMail("$emailprefix $user.username pushed $commitCount commits => $repository.name", "$summaryUrl\n$changes", toAddresses) \ No newline at end of file -- Gitblit v1.9.1