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

Install nodeJSScan tool for static security scan #57

Open
wants to merge 1 commit into
base: f24
Choose a base branch
from

Conversation

tatougueye
Copy link

@tatougueye tatougueye commented Oct 27, 2024

Integrated nodeJSScan a static security code scanner for Node.js applications. This tool detects and gives you a rundown of security issues you might have with a specific file and if not issues are detected it will indicate that. Used the CLI approach by first running pip install njsscan, after running njsscan. To see a security analysis of a js file, you have to run on the terminal njsscan <filepath>.
Screen Shot 2024-10-26 at 5 10 16 AM
Screen Shot 2024-10-26 at 5 10 26 AM

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11542720950

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 82.68%

Totals Coverage Status
Change from base Build 11542692737: 0.005%
Covered Lines: 22363
Relevant Lines: 25627

💛 - Coveralls

@mohamed-elzeni mohamed-elzeni changed the title added a static security tool Install nodeJSScan tool for static security scan Oct 27, 2024
@mohamed-elzeni mohamed-elzeni added the enhancement New feature or request label Oct 27, 2024
@mohamed-elzeni mohamed-elzeni added this to the P3A milestone Oct 27, 2024
@mohamed-elzeni
Copy link
Member

I have tested the tool before. Look good to me!

Copy link

@lhitmi lhitmi left a comment

Choose a reason for hiding this comment

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

I have tried this tool before, this looks good

@mohamed-elzeni mohamed-elzeni linked an issue Oct 27, 2024 that may be closed by this pull request
@lhitmi lhitmi self-requested a review October 27, 2024 18:28
Copy link

@lhitmi lhitmi left a comment

Choose a reason for hiding this comment

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

it looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Install static analysis tool for codebase
4 participants