Mogrus initialization fix, gitea PRs, regexp fix.

Fixed Mogrus initialization to pass log level. Configuration to
follow.

OpenSAPS now fully working with Gitea's pull requests.

Fixed URL/URL names regexp for Gitlab and Gitea to include "+" in
them.
This commit is contained in:
Stanislav Nikitin 2017-11-11 13:00:50 +05:00
parent c9a111c832
commit e969b7018d
3 changed files with 443 additions and 290 deletions

View File

@ -50,7 +50,7 @@ func (c *Context) Initialize() {
l := mogrus.New() l := mogrus.New()
l.Initialize() l.Initialize()
c.Log = l.CreateLogger("opensaps") c.Log = l.CreateLogger("opensaps")
c.Log.CreateOutput("stdout", os.Stdout, true) c.Log.CreateOutput("stdout", os.Stdout, true, "debug")
c.Flagger = flagger.New(c.Log) c.Flagger = flagger.New(c.Log)
c.Flagger.Initialize() c.Flagger.Initialize()

View File

@ -34,10 +34,28 @@ func (gp GiteaParser) Initialize() {
c.Log.Infoln("Initializing Gitea parser...") c.Log.Infoln("Initializing Gitea parser...")
} }
func (gp GiteaParser) parseBranchNew(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo} | Branch: {branch}] {header_message}."
// Parse header.
// [0] is repo, [1] is branch.
header_data := gp.cutLinks(message.Text)
data["repo"] = header_data[0][1]
data["repo_url"] = header_data[0][0]
data["branch"] = header_data[1][1]
data["branch_url"] = header_data[1][0]
header_msg := strings.Split(message.Text, "] ")[1]
data["header_message"] = header_msg
return data
}
func (gp GiteaParser) cutLinks(data string) [][]string { func (gp GiteaParser) cutLinks(data string) [][]string {
c.Log.Debugln("Passed:", data) c.Log.Debugln("Passed:", data)
r := regexp.MustCompile("((https??://[a-zA-Z0-9.#!*/ _-]+)\\|([a-zA-Z0-9.#!*/ _-]+))") r := regexp.MustCompile("((https??://[a-zA-Z0-9.#!*/ _-]+)\\|([a-zA-Z0-9.#!*/ _+-]+))")
found := r.FindAllStringSubmatch(data, -1) found := r.FindAllStringSubmatch(data, -1)
@ -93,6 +111,20 @@ func (gp GiteaParser) ParseMessage(message slackmessage.SlackMessage) map[string
var data map[string]string var data map[string]string
if strings.Contains(message.Text, "new commit") && strings.Contains(message.Text, "pushed by ") { if strings.Contains(message.Text, "new commit") && strings.Contains(message.Text, "pushed by ") {
data = gp.parseCommitNew(message) data = gp.parseCommitNew(message)
} else if strings.Contains(message.Text, "Pull request closed") {
data = gp.parsePullRequestClosed(message)
} else if strings.Contains(message.Text, "Pull request edited") {
data = gp.parsePullRequestEdited(message)
} else if strings.Contains(message.Text, "Pull request labels updated") {
data = gp.parsePullRequestLabelsUpdated(message)
} else if strings.Contains(message.Text, "Pull request merged") {
data = gp.parsePullRequestMerged(message)
} else if strings.Contains(message.Text, "Pull request re-opened") {
data = gp.parsePullRequestReopened(message)
} else if strings.Contains(message.Text, "Pull request submitted") {
data = gp.parsePullRequestSubmitted(message)
} else if strings.Contains(message.Text, "branch created by") {
data = gp.parseBranchNew(message)
} else { } else {
return map[string]string{"message": "Unknown message type:<br />" + fmt.Sprintf("%+v", message)} return map[string]string{"message": "Unknown message type:<br />" + fmt.Sprintf("%+v", message)}
} }
@ -101,3 +133,124 @@ func (gp GiteaParser) ParseMessage(message slackmessage.SlackMessage) map[string
return data return data
} }
func (gp GiteaParser) parsePullRequestClosed(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo}] Pull request {pr} by {user} was closed."
// Parse header.
// [0] is repo, [1] is branch.
header_data := gp.cutLinks(message.Text)
data["pr"] = header_data[0][1]
data["pr_url"] = header_data[0][0]
data["user"] = header_data[1][1]
data["user_url"] = header_data[1][0]
repo := strings.Split(message.Text, "Pull request")[0]
repo = strings.TrimLeft(repo, "[")
repo = strings.TrimRight(repo, "] ")
data["repo"] = repo
return data
}
func (gp GiteaParser) parsePullRequestEdited(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo}] Pull request {pr} was edited by {user}."
// Parse header.
// [0] is repo, [1] is branch.
header_data := gp.cutLinks(message.Text)
data["pr"] = header_data[0][1]
data["pr_url"] = header_data[0][0]
data["user"] = header_data[1][1]
data["user_url"] = header_data[1][0]
repo := strings.Split(message.Text, "Pull request")[0]
repo = strings.TrimLeft(repo, "[")
repo = strings.TrimRight(repo, "] ")
data["repo"] = repo
return data
}
func (gp GiteaParser) parsePullRequestLabelsUpdated(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo}] Pull request {pr} by {user} labels was updated."
// Parse header.
// [0] is repo, [1] is branch.
header_data := gp.cutLinks(message.Text)
data["pr"] = header_data[0][1]
data["pr_url"] = header_data[0][0]
data["user"] = header_data[1][1]
data["user_url"] = header_data[1][0]
repo := strings.Split(message.Text, "Pull request")[0]
repo = strings.TrimLeft(repo, "[")
repo = strings.TrimRight(repo, "] ")
data["repo"] = repo
return data
}
func (gp GiteaParser) parsePullRequestMerged(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo}] Pull request {pr} by {user} was merged."
// Parse header.
// [0] is repo, [1] is branch.
header_data := gp.cutLinks(message.Text)
data["pr"] = header_data[0][1]
data["pr_url"] = header_data[0][0]
data["user"] = header_data[1][1]
data["user_url"] = header_data[1][0]
repo := strings.Split(message.Text, "Pull request")[0]
repo = strings.TrimLeft(repo, "[")
repo = strings.TrimRight(repo, "] ")
data["repo"] = repo
return data
}
func (gp GiteaParser) parsePullRequestReopened(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo}] Pull request {pr} by {user} was re-opened."
// Parse header.
// [0] is repo, [1] is branch.
header_data := gp.cutLinks(message.Text)
data["pr"] = header_data[0][1]
data["pr_url"] = header_data[0][0]
data["user"] = header_data[1][1]
data["user_url"] = header_data[1][0]
repo := strings.Split(message.Text, "Pull request")[0]
repo = strings.TrimLeft(repo, "[")
repo = strings.TrimRight(repo, "] ")
data["repo"] = repo
return data
}
func (gp GiteaParser) parsePullRequestSubmitted(message slackmessage.SlackMessage) map[string]string {
data := make(map[string]string)
data["message"] = "[Repo: {repo}] Pull request {pr} submitted by {user}."
header_data := gp.cutLinks(message.Text)
data["user"] = header_data[0][1]
data["user_url"] = header_data[0][0]
repo := strings.Split(message.Text, "Pull request")[0]
repo = strings.TrimLeft(repo, "[")
repo = strings.TrimRight(repo, "] ")
data["repo"] = repo
// There is a pull request description in attachments.
prdata := gp.cutLinks(message.Attachments[0].Title)
data["pr"] = prdata[0][1]
data["pr_url"] = prdata[0][0]
return data
}

View File

@ -31,7 +31,7 @@ import (
type GitlabParser struct{} type GitlabParser struct{}
func (gp GitlabParser) cutLinks(data string) [][]string { func (gp GitlabParser) cutLinks(data string) [][]string {
r := regexp.MustCompile("((https??://[a-zA-Z0-9.#!*/ _-]+)\\|([a-zA-Z0-9.#!*/ _-]+))") r := regexp.MustCompile("((https??://[a-zA-Z0-9.#!*/ _-]+)\\|([a-zA-Z0-9.#!*/ _+-]+))")
found := r.FindAllStringSubmatch(data, -1) found := r.FindAllStringSubmatch(data, -1)