CheckAndEnsureSafePR was ignored when merging

harryzcy · harryzcy · commit acd0ad220fcf · 2023-04-06T12:35:58.000-04:00
diff --git a/services/migrations/github.go b/services/migrations/github.go
@@ -841,6 +841,9 @@ func (g *GithubDownloaderV3) GetNewPullRequests(page, perPage int, updatedAfter
 			return nil, false, err
 		}
 		allPRs = append(allPRs, basePR)
+
+		// SECURITY: Ensure that the PR is safe
+		_ = CheckAndEnsureSafePR(allPRs[len(allPRs)-1], g.baseURL, g)
 	}
 
 	return allPRs, len(issues) < perPage, nil

Original file line number	Diff line number	Diff line change
`@@ -841,6 +841,9 @@ func (g *GithubDownloaderV3) GetNewPullRequests(page, perPage int, updatedAfter`
`841`	`841`	`return nil, false, err`
`842`	`842`	`}`
`843`	`843`	`allPRs = append(allPRs, basePR)`
	`844`	`+`
	`845`	`+ // SECURITY: Ensure that the PR is safe`
	`846`	`+ _ = CheckAndEnsureSafePR(allPRs[len(allPRs)-1], g.baseURL, g)`
`844`	`847`	`}`
`845`	`848`
`846`	`849`	`return allPRs, len(issues) < perPage, nil`