Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Partial implementation of template parameter & report #9

Open
wants to merge 40 commits into
base: main
Choose a base branch
from

Conversation

jorge-abarca
Copy link

@jorge-abarca jorge-abarca commented Apr 20, 2022

This PR is meant to help with a partial implementation of handling the template parameter based on the work done by GeekMasher.

This includes a pretty basic report template that breaks open issues by severity. It also includes an action for updating the dist folder, as well as some PRs suggested by Dependabot.

dependabot bot and others added 22 commits April 19, 2022 20:35
Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

---
updated-dependencies:
- dependency-name: minimist
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [ansi-regex](https://github.com/chalk/ansi-regex) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/chalk/ansi-regex/releases)
- [Commits](chalk/ansi-regex@v3.0.0...v3.0.1)

---
updated-dependencies:
- dependency-name: ansi-regex
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.1 to 2.6.7.
- [Release notes](https://github.com/node-fetch/node-fetch/releases)
- [Commits](node-fetch/node-fetch@v2.6.1...v2.6.7)

---
updated-dependencies:
- dependency-name: node-fetch
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [glob-parent](https://github.com/gulpjs/glob-parent) from 5.1.1 to 5.1.2.
- [Release notes](https://github.com/gulpjs/glob-parent/releases)
- [Changelog](https://github.com/gulpjs/glob-parent/blob/main/CHANGELOG.md)
- [Commits](gulpjs/glob-parent@v5.1.1...v5.1.2)

---
updated-dependencies:
- dependency-name: glob-parent
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [pathval](https://github.com/chaijs/pathval) from 1.1.0 to 1.1.1.
- [Release notes](https://github.com/chaijs/pathval/releases)
- [Changelog](https://github.com/chaijs/pathval/blob/master/CHANGELOG.md)
- [Commits](chaijs/pathval@v1.1.0...v1.1.1)

---
updated-dependencies:
- dependency-name: pathval
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [ws](https://github.com/websockets/ws) from 7.4.2 to 7.5.7.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](websockets/ws@7.4.2...7.5.7)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
…al-1.1.1

Bump pathval from 1.1.0 to 1.1.1
…parent-5.1.2

Bump glob-parent from 5.1.1 to 5.1.2
…fetch-2.6.7

Bump node-fetch from 2.6.1 to 2.6.7
…regex-3.0.1

Bump ansi-regex from 3.0.0 to 3.0.1
…ist-1.2.6

Bump minimist from 1.2.5 to 1.2.6
Added CodeQL to perform static code analysis
@jorge-abarca
Copy link
Author

I made additional changes to the PR. Let me list what they are.

Bug Fixes

This PR addresses the following issues:

New Functionality

This PR also adds the following functionality:

  • Severity has been expanded to include critical, high and low, as presented by GitHub's UI.
  • The summary report shows the new severity classifications with their number of incidences.
  • An alerts component has been created to be available for reports.
  • The report template has been modified to include all open alerts grouped by severity.
  • An aggregated_report template has been included to create less verbose reports showing the number of instances a rule was matched.

Development Experience Improvements

Additionally, this PR has the following proposed development experience improvements:

  • The "Build and Test" action uploads a "results" artifact for debugging purposes.
  • Mocking is introduced to allow tests to work without the need to have access to any repositories.

Mocking is achieved by leveraging a mocking directory inside samples. JSON files representing the mocked responses for the paginate and graphql methods are used by testUtils.js to provide this functionality.

dependabot bot and others added 6 commits August 18, 2022 19:55
Bumps [@actions/core](https://github.com/actions/toolkit/tree/HEAD/packages/core) from 1.2.6 to 1.9.1.
- [Release notes](https://github.com/actions/toolkit/releases)
- [Changelog](https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md)
- [Commits](https://github.com/actions/toolkit/commits/HEAD/packages/core)

---
updated-dependencies:
- dependency-name: "@actions/core"
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
…ns/core-1.9.1

Bump @actions/core from 1.2.6 to 1.9.1
Fixed a `sarifRules.push is not iterable` error raised when having multiple languages and one of them has no rules defined.
Bumps [decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) from 0.2.0 to 0.2.2.
- [Release notes](https://github.com/SamVerschueren/decode-uri-component/releases)
- [Commits](SamVerschueren/decode-uri-component@v0.2.0...v0.2.2)

---
updated-dependencies:
- dependency-name: decode-uri-component
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
jorge-abarca and others added 4 commits January 11, 2023 17:55
Fixed the never-ending loop caused by using `parameters` instead of `queryParameters`. If vulnerabilities or dependencies had a second page the action would take as long as the access token's expiration.
Modified the puppeteer's launch method to prevent this to hang in Linux, as per: https://stackoverflow.com/a/61050539
jorge-abarca and others added 8 commits January 12, 2023 16:19
…e-uri-component-0.2.2

Bump decode-uri-component from 0.2.0 to 0.2.2
Updated runs to specify node16.
Will create another branch to test upgrading to node 16.
Bumps [minimatch](https://github.com/isaacs/minimatch) to 3.1.2 and updates ancestor dependency [mocha](https://github.com/mochajs/mocha). These dependencies need to be updated together.


Updates `minimatch` from 3.0.4 to 3.1.2
- [Release notes](https://github.com/isaacs/minimatch/releases)
- [Changelog](https://github.com/isaacs/minimatch/blob/main/changelog.md)
- [Commits](isaacs/minimatch@v3.0.4...v3.1.2)

Updates `mocha` from 8.2.1 to 10.2.0
- [Release notes](https://github.com/mochajs/mocha/releases)
- [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md)
- [Commits](mochajs/mocha@v8.2.1...v10.2.0)

---
updated-dependencies:
- dependency-name: minimatch
  dependency-type: indirect
- dependency-name: mocha
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Changed the action configuration to run using node 16, the underlying code would need to be upgraded but this removes the deprecation warning in the pipelines using it.
…atch-and-mocha-3.1.2

Bump minimatch and mocha
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.

1 participant