From 915d3525054470e997e255fef88c5f6642e1a4b0 Mon Sep 17 00:00:00 2001 From: syuilo Date: Fri, 23 Nov 2018 16:39:51 +0900 Subject: [PATCH] Resolve #3366 --- src/config/types.ts | 5 - src/server/api/index.ts | 1 - src/server/api/service/github-bot.ts | 163 --------------------------- 3 files changed, 169 deletions(-) delete mode 100644 src/server/api/service/github-bot.ts diff --git a/src/config/types.ts b/src/config/types.ts index a08b90fec3..3d78685e6a 100644 --- a/src/config/types.ts +++ b/src/config/types.ts @@ -39,11 +39,6 @@ export type Source = { accesslog?: string; - github_bot?: { - hook_secret: string; - username: string; - }; - /** * Service Worker */ diff --git a/src/server/api/index.ts b/src/server/api/index.ts index 1cd0028574..87da490acb 100644 --- a/src/server/api/index.ts +++ b/src/server/api/index.ts @@ -45,7 +45,6 @@ router.post('/signin', require('./private/signin').default); router.use(require('./service/discord').routes()); router.use(require('./service/github').routes()); -router.use(require('./service/github-bot').routes()); router.use(require('./service/twitter').routes()); router.use(require('./mastodon').routes()); diff --git a/src/server/api/service/github-bot.ts b/src/server/api/service/github-bot.ts deleted file mode 100644 index c201736673..0000000000 --- a/src/server/api/service/github-bot.ts +++ /dev/null @@ -1,163 +0,0 @@ -import * as EventEmitter from 'events'; -import * as Router from 'koa-router'; -import * as request from 'request'; -import User, { IUser } from '../../../models/user'; -import createNote from '../../../services/note/create'; -import config from '../../../config'; -const crypto = require('crypto'); - -const handler = new EventEmitter(); - -let bot: IUser; - -const post = async (text: string, home = true) => { - if (bot == null) { - const account = await User.findOne({ - usernameLower: config.github_bot.username.toLowerCase() - }); - - if (account == null) { - console.warn(`GitHub hook bot specified, but not found: @${config.github_bot.username}`); - return; - } else { - bot = account; - } - } - - createNote(bot, { text, visibility: home ? 'home' : 'public' }); -}; - -// Init router -const router = new Router(); - -if (config.github_bot) { - const secret = config.github_bot.hook_secret; - - router.post('/hooks/github', ctx => { - const body = JSON.stringify(ctx.request.body); - const hash = crypto.createHmac('sha1', secret).update(body).digest('hex'); - const sig1 = new Buffer(ctx.headers['x-hub-signature']); - const sig2 = new Buffer(`sha1=${hash}`); - - // シグネチャ比較 - if (sig1.equals(sig2)) { - handler.emit(ctx.headers['x-github-event'], ctx.request.body); - ctx.status = 204; - } else { - ctx.status = 400; - } - }); -} - -module.exports = router; - -handler.on('status', event => { - const state = event.state; - switch (state) { - case 'error': - case 'failure': - const commit = event.commit; - const parent = commit.parents[0]; - - // Fetch parent status - request({ - url: `${parent.url}/statuses`, - proxy: config.proxy, - headers: { - 'User-Agent': 'misskey' - } - }, (err, res, body) => { - if (err) { - console.error(err); - return; - } - const parentStatuses = JSON.parse(body); - const parentState = parentStatuses[0].state; - const stillFailed = parentState == 'failure' || parentState == 'error'; - if (stillFailed) { - post(`⚠️**BUILD STILL FAILED**⚠️: ?[${commit.commit.message}](${commit.html_url})`); - } else { - post(`🚨**BUILD FAILED**🚨: →→→?[${commit.commit.message}](${commit.html_url})←←←`); - } - }); - break; - } -}); - -handler.on('push', event => { - const ref = event.ref; - switch (ref) { - case 'refs/heads/develop': - const pusher = event.pusher; - const compare = event.compare; - const commits: any[] = event.commits; - post([ - `🆕 Pushed by **${pusher.name}** with ?[${commits.length} commit${commits.length > 1 ? 's' : ''}](${compare}):`, - commits.reverse().map(commit => `・[?[${commit.id.substr(0, 7)}](${commit.url})] ${commit.message.split('\n')[0]}`).join('\n'), - ].join('\n')); - break; - } -}); - -handler.on('issues', event => { - const issue = event.issue; - const action = event.action; - let title: string; - switch (action) { - case 'opened': title = '💥 Issue opened'; break; - case 'closed': title = '💮 Issue closed'; break; - case 'reopened': title = '🔥 Issue reopened'; break; - default: return; - } - post(`${title}: <${issue.number}>「${issue.title}」\n${issue.html_url}`); -}); - -handler.on('issue_comment', event => { - const issue = event.issue; - const comment = event.comment; - const action = event.action; - let text: string; - switch (action) { - case 'created': text = `💬 Commented to「${issue.title}」:${comment.user.login}「${comment.body}」\n${comment.html_url}`; break; - default: return; - } - post(text); -}); - -handler.on('release', event => { - const action = event.action; - const release = event.release; - let text: string; - switch (action) { - case 'published': text = `🎁 **NEW RELEASE**: [${release.tag_name}](${release.html_url}) is out now. Enjoy!`; break; - default: return; - } - post(text); -}); - -handler.on('watch', event => { - const sender = event.sender; - post(`(((⭐️))) Starred by **${sender.login}** (((⭐️)))`, false); -}); - -handler.on('fork', event => { - const repo = event.forkee; - post(`🍴 Forked:\n${repo.html_url} 🍴`); -}); - -handler.on('pull_request', event => { - const pr = event.pull_request; - const action = event.action; - let text: string; - switch (action) { - case 'opened': text = `📦 New Pull Request:「${pr.title}」\n${pr.html_url}`; break; - case 'reopened': text = `🗿 Pull Request Reopened:「${pr.title}」\n${pr.html_url}`; break; - case 'closed': - text = pr.merged - ? `💯 Pull Request Merged!:「${pr.title}」\n${pr.html_url}` - : `🚫 Pull Request Closed:「${pr.title}」\n${pr.html_url}`; - break; - default: return; - } - post(text); -});