actions
/
stale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

not print untrusted info to STDOUT.

This commit is contained in:
TingluoHuang 2020-11-02 16:43:44 -05:00
parent 707ba4d103
commit 658139a1f1
2 changed files with 12 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
dist/index.js vendored
View File

@ -75,7 +75,7 @@ class IssueProcessor {
} }
for (const issue of issues.values()) { for (const issue of issues.values()) {
const isPr = !!issue.pull_request; const isPr = !!issue.pull_request;
core.info(`Found issue: issue #${issue.number} - ${issue.title} last updated ${issue.updated_at} (is pr? ${isPr})`); core.info(`Found issue: issue #${issue.number} last updated ${issue.updated_at} (is pr? ${isPr})`);
// calculate string based messages for this issue // calculate string based messages for this issue
const staleMessage = isPr const staleMessage = isPr
? this.options.stalePrMessage ? this.options.stalePrMessage
@ -172,7 +172,7 @@ class IssueProcessor {
// find any comments since the date // find any comments since the date
const comments = yield this.listIssueComments(issue.number, sinceDate); const comments = yield this.listIssueComments(issue.number, sinceDate);
const filteredComments = comments.filter(comment => comment.user.type === 'User' && comment.user.login !== github_1.context.actor); const filteredComments = comments.filter(comment => comment.user.type === 'User' && comment.user.login !== github_1.context.actor);
core.info(`Comments not made by ${github_1.context.actor} or another bot: ${filteredComments.length}`); core.info(`Comments not made by actor or another bot: ${filteredComments.length}`);
// if there are any user comments returned // if there are any user comments returned
return filteredComments.length > 0; return filteredComments.length > 0;
}); });
@ -222,7 +222,7 @@ class IssueProcessor {
// Mark an issue as stale with a comment and a label // Mark an issue as stale with a comment and a label
markStale(issue, staleMessage, staleLabel, skipMessage) { markStale(issue, staleMessage, staleLabel, skipMessage) {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
core.info(`Marking issue #${issue.number} - ${issue.title} as stale`); core.info(`Marking issue #${issue.number} as stale`);
this.staleIssues.push(issue); this.staleIssues.push(issue);
this.operationsLeft -= 2; this.operationsLeft -= 2;
// if the issue is being marked stale, the updated date should be changed to right now // if the issue is being marked stale, the updated date should be changed to right now
@ -261,7 +261,7 @@ class IssueProcessor {
// Close an issue based on staleness // Close an issue based on staleness
closeIssue(issue, closeMessage, closeLabel) { closeIssue(issue, closeMessage, closeLabel) {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
core.info(`Closing issue #${issue.number} - ${issue.title} for being stale`); core.info(`Closing issue #${issue.number} for being stale`);
this.closedIssues.push(issue); this.closedIssues.push(issue);
this.operationsLeft -= 1; this.operationsLeft -= 1;
if (this.options.debugOnly) { if (this.options.debugOnly) {
@ -309,7 +309,7 @@ class IssueProcessor {
// Remove a label from an issue // Remove a label from an issue
removeLabel(issue, label) { removeLabel(issue, label) {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
core.info(`Removing label ${label} from issue #${issue.number} - ${issue.title}`); core.info(`Removing label from issue #${issue.number}`);
this.removedLabelIssues.push(issue); this.removedLabelIssues.push(issue);
this.operationsLeft -= 1; this.operationsLeft -= 1;
if (this.options.debugOnly) { if (this.options.debugOnly) {
@ -332,7 +332,7 @@ class IssueProcessor {
///see https://developer.github.com/v3/activity/events/ ///see https://developer.github.com/v3/activity/events/
getLabelCreationDate(issue, label) { getLabelCreationDate(issue, label) {
return __awaiter(this, void 0, void 0, function* () { return __awaiter(this, void 0, void 0, function* () {
core.info(`Checking for label ${label} on issue #${issue.number}`); core.info(`Checking for label on issue #${issue.number}`);
this.operationsLeft -= 1; this.operationsLeft -= 1;
const options = this.client.issues.listEvents.endpoint.merge({ const options = this.client.issues.listEvents.endpoint.merge({
owner: github_1.context.repo.owner, owner: github_1.context.repo.owner,

16
src/IssueProcessor.ts
View File

@ -115,7 +115,7 @@ export class IssueProcessor {
const isPr = !!issue.pull_request; const isPr = !!issue.pull_request;
core.info( core.info(
`Found issue: issue #${issue.number} - ${issue.title} last updated ${issue.updated_at} (is pr? ${isPr})` `Found issue: issue #${issue.number} last updated ${issue.updated_at} (is pr? ${isPr})`
); );
// calculate string based messages for this issue // calculate string based messages for this issue
@ -277,7 +277,7 @@ export class IssueProcessor {
); );
core.info( core.info(
`Comments not made by ${context.actor} or another bot: ${filteredComments.length}` `Comments not made by actor or another bot: ${filteredComments.length}`
); );
// if there are any user comments returned // if there are any user comments returned
@ -336,7 +336,7 @@ export class IssueProcessor {
staleLabel: string, staleLabel: string,
skipMessage: boolean skipMessage: boolean
): Promise<void> { ): Promise<void> {
core.info(`Marking issue #${issue.number} - ${issue.title} as stale`); core.info(`Marking issue #${issue.number} as stale`);
this.staleIssues.push(issue); this.staleIssues.push(issue);
@ -382,9 +382,7 @@ export class IssueProcessor {
closeMessage?: string, closeMessage?: string,
closeLabel?: string closeLabel?: string
): Promise<void> { ): Promise<void> {
core.info( core.info(`Closing issue #${issue.number} for being stale`);
`Closing issue #${issue.number} - ${issue.title} for being stale`
);
this.closedIssues.push(issue); this.closedIssues.push(issue);
@ -434,9 +432,7 @@ export class IssueProcessor {
// Remove a label from an issue // Remove a label from an issue
private async removeLabel(issue: Issue, label: string): Promise<void> { private async removeLabel(issue: Issue, label: string): Promise<void> {
core.info( core.info(`Removing label from issue #${issue.number}`);
`Removing label ${label} from issue #${issue.number} - ${issue.title}`
);
this.removedLabelIssues.push(issue); this.removedLabelIssues.push(issue);
@ -464,7 +460,7 @@ export class IssueProcessor {
issue: Issue, issue: Issue,
label: string label: string
): Promise<string | undefined> { ): Promise<string | undefined> {
core.info(`Checking for label ${label} on issue #${issue.number}`); core.info(`Checking for label on issue #${issue.number}`);
this.operationsLeft -= 1; this.operationsLeft -= 1;