Mailout the PR comments

salathe · Jul 31, 2014 · f006fcf · f006fcf
1 parent 12e6346
commit f006fcf
Showing 1 changed file with 59 additions and 25 deletions.
diff --git a/github-webhook.php b/github-webhook.php
@@ -1,6 +1,42 @@
 <?php
-$config = array(
+function verify_signature($requestBody) {
+	if(isset($_SERVER['HTTP_X_HUB_SIGNATURE'])){
+		$parts = explode("=", $_SERVER['HTTP_X_HUB_SIGNATURE'], 2);
+		if (count($parts) == 2) {
+			return hash_hmac($parts[0], $requestBody, getenv('GITHUB_SECRET')) === $parts[1];
+		}
+	}
+	return false;
+}
+
+function get_repo_email($repos, $repoName) {
+    // if we somehow end up receiving a PR for a repo not matching anything send it to systems so that we can fix it
+    $to = 'systems@php.net';
+    foreach ($repos as $repoPrefix => $email) {
+        if (strpos($repoName, $repoPrefix) === 0) {
+            $to = $email;
+        }
+    }
+
+    return $to;
+}
+
+function prep_title($action, $PR, $base) {
+    $PRNumber = $PR->number;
+    $title = $PR->title;
+
+    $repoName = $base->repo->name;
+    $targetBranch = $base->ref;
+
+    $subject = sprintf('[PR][%s][#%s][%s][%s] - %s', $repoName, $PRNumber, $targetBranch, $action, $title);
+
+    return $subject;
+}
+
+
+$CONFIG = array(
 	'repos' => array(
+		'php-langspec' => 'standards@lists.php.net',
 		'php-src' => 'git-pulls@lists.php.net',
 		'web-' => 'php-webmaster@lists.php.net',
 		'pecl-' => 'pecl-dev@lists.php.net',
@@ -20,25 +56,14 @@
 	case 'pull_request':
 		$payload = json_decode($body);
 		$action = $payload->action;
-		$PRNumber = $payload->number;
 		$PR = $payload->pull_request;
-		$htmlUrl = $PR->html_url;
-		$title = $PR->title;
-		$description = $PR->body;
-		$repoName = $PR->base->repo->name;
-
-		$targetBranch = $PR->base->ref;
 		$mergeable = $PR->mergeable;
+        $htmlUrl = $PR->html_url;
+        $description = $PR->body;
 
-		// if we somehow end up receiving a PR for a repo not matching anything send it to systems so that we can fix it
-		$to = 'systems@php.net';
-		foreach ($config['repos'] as $repoPrefix => $email) {
-			if (strpos($repoName, $repoPrefix) === 0) {
-				$to = $email;
-			}
-		}
+        $to = get_repo_email($CONFIG["repos"], $repoName);
+        $subject = prep_title($action, $PR, $PR->base);
 
-		$subject = sprintf('[PR][%s][#%s][%s][%s] - %s', $repoName, $PRNumber, $targetBranch, $action, $title);
 		$message = sprintf("You can view the Pull Request on github:\r\n%s", $htmlUrl);
 		if ($mergeable === false) {
 			$message .= "\r\n\r\nWarning: according to github, the Pull Request cannot be merged without manual conflict resolution!";
@@ -47,16 +72,25 @@
 		$headers = "From: noreply@php.net\r\nContent-Type: text/plain; charset=utf-8\r\n";
 		mail($to, '=?utf-8?B?'.base64_encode($subject).'?=', $message, $headers, "-fnoreply@php.net");
 		break;
+
+    case 'pull_request_review_comment':
+		$payload = json_decode($body);
+		$action = $payload->action;
+		$PR = $payload->pull_request;
+        $username = $payload->user->login;
+		$comment = $payload->comment->body;
+
+        $to = get_repo_email($CONFIG["repos"], $repoName);
+        $subject = prep_title($action, $PR, $PR->base);
+		$message = sprintf("You can view the Pull Request on github:\r\n%s", $htmlUrl);
+		$message .= sprintf("\r\n\r\nPull Request Comment:\r\n%s", $description);
+		$message .= sprintf("\r\nMade by: %s", $username);
+
+		$headers = "From: noreply@php.net\r\nContent-Type: text/plain; charset=utf-8\r\n";
+		mail($to, '=?utf-8?B?'.base64_encode($subject).'?=', $message, $headers, "-fnoreply@php.net");
+        break;
+
 	default:
 		header('HTTP/1.1 501 Not Implemented');
 }
 
-function verify_signature($requestBody) {
-	if(isset($_SERVER['HTTP_X_HUB_SIGNATURE'])){
-		$parts = explode("=", $_SERVER['HTTP_X_HUB_SIGNATURE'], 2);
-		if (count($parts) == 2) {
-			return hash_hmac($parts[0], $requestBody, getenv('GITHUB_SECRET')) === $parts[1];
-		}
-	}
-	return false;
-}