Skip to content

1#1

Merged
dotam99 merged 508 commits into
dotam99:mainfrom
github:main
Apr 14, 2021
Merged

1#1
dotam99 merged 508 commits into
dotam99:mainfrom
github:main

Conversation

@dotam99

@dotam99 dotam99 commented Apr 8, 2021

Copy link
Copy Markdown
Owner

Why:

Closes [issue link]

What's being changed:

Check off the following:

@guardrails

guardrails Bot commented Apr 8, 2021

Copy link
Copy Markdown

⚠️ We detected 7 security issues in this pull request:

Insecure File Management (1)

const schemas = files.map(filename => require(filename))

More info on how to fix Insecure File Management in Javascript.


Insecure Use of Dangerous Function (3)

const { execSync } = require('child_process')

const exec = promisify(require('child_process').exec)

const { execSync } = require('child_process')

More info on how to fix Insecure Use of Dangerous Function in Javascript.


Vulnerable Libraries (3)

More info on how to fix Vulnerable Libraries in Javascript.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

Octomerger and others added 27 commits April 9, 2021 06:27
Add move reopened issues to triage column workflow
…18328)

Co-authored-by: Jason Etcovitch <jasonetco@github.com>
Co-authored-by: Rachael Sewell <rachmari@github.com>
Co-authored-by: Sarah Schneider <sarahs@users.noreply.github.com>
* Revised quickstart example, added screenshots

* Updated example

* Updated result and example

* Slight re-wording

* Added explanations for each action

* Removed superfluous bits

* Update actions-quickstart-workflow-sidebar.png

* Update actions-quickstart-job.png

* Update actions-quickstart-job.png

* Update actions-quickstart-job.png

* Update quickstart.md

* Apply suggestions from code review

Co-authored-by: Lucas Costi <lucascosti@users.noreply.github.com>

* Apply suggestions from code review

* Added details about ls action

Co-authored-by: Lucas Costi <lucascosti@users.noreply.github.com>
…c` tool (#18631)

Co-authored-by: Laura Coursen <lecoursen@github.com>

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.