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

Add Snyk for Vulns and License checks #293

Merged
merged 1 commit into from
Nov 18, 2023
Merged

Conversation

leordev
Copy link
Contributor

@leordev leordev commented Nov 14, 2023

  • add security scanning to detect vulns
  • add license check
  • upload report to Github Code Scanning

Fix #280
Fix #281

Copy link

codesandbox bot commented Nov 14, 2023

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@leordev leordev force-pushed the leordev/issue-280-281.snyk branch from 3f31b6b to 013e5ab Compare November 14, 2023 21:56
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link
Contributor

github-actions bot commented Nov 14, 2023

TBDocs Report

✅ No errors or warnings

@web5/api

  • Project entry file: packages/api/src/index.ts

Updated @ 2023-11-14T21:59:44.788Z - Commit: f681bd4

@leordev leordev marked this pull request as ready for review November 14, 2023 21:57
Copy link

codecov bot commented Nov 14, 2023

Codecov Report

Merging #293 (013e5ab) into main (2421b28) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #293   +/-   ##
=======================================
  Coverage   90.34%   90.34%           
=======================================
  Files          74       74           
  Lines       14028    14028           
  Branches     1382     1382           
=======================================
  Hits        12674    12674           
  Misses       1328     1328           
  Partials       26       26           
Components Coverage Δ
api 95.04% <ø> (ø)
common 95.00% <ø> (ø)
credentials 94.49% <ø> (ø)
crypto 100.00% <ø> (ø)
dids 88.75% <ø> (ø)
agent 87.22% <ø> (ø)
identity-agent 56.81% <ø> (ø)
proxy-agent 58.43% <ø> (ø)
user-agent 55.22% <ø> (ø)

Copy link
Contributor

@mistermoe mistermoe left a comment

Choose a reason for hiding this comment

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

awesome @leordev !

@frankhinek frankhinek merged commit fb61e3b into main Nov 18, 2023
26 of 27 checks passed
@frankhinek frankhinek deleted the leordev/issue-280-281.snyk branch November 18, 2023 17:12
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.

Setup Security Scanning Setup License Checks
3 participants