Closed pnijhara closed 6 months ago
cc @taoufik07 @ken-reitz What do you think?
Hey @pnijhara, thanks for this tool ! for now let's just create PRs to fix the potential code quality improvements, the security and bug issues are on high priority.
Sure! I will open issues for discussion on security and bug-risk issues.
I'm going to close this, for tracking
I can send a PR with some fixes if you want.
that would be great!
@kennethreitz do you want me to work on this?
sure!
I forked this repo and ran a DeepSource analysis over it. There:
31 Security issues were found
Out of which 22 can be auto fixed. Some of them can be found here
https://deepsource.io/gh/pnijhara/responder/issue/BAN-B101/
In all total number of issues are listed here - https://deepsource.io/gh/pnijhara/responder/
Link to
.deepsource.toml
configuration file that was added to the forked repo for running DeepSource analysis can be found hereWhich of these issues you would consider high priority, and which ones would you want to ignore or consider false-positive?
I can send a PR with some fixes if you want. You can also activate this analysis on your own, and have it triggered on each PR if you integrate DeepSource in the repo
Disclaimer: I work for DeepSource