-
-
Notifications
You must be signed in to change notification settings - Fork 278
Add support for parsing Git commit messages #1992
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
Conversation
06e8b82 to
976c5ad
Compare
|
|
||
| def clone(self): | ||
| """Clone the repository.""" | ||
| self.repo_url = "https://github.com/torvalds/linux" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This part should not be static
keshav-space
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ziadhany, see some suggestions.
vulnerabilities/pipelines/v2_importers/collect_repo_fix_commits.py
Outdated
Show resolved
Hide resolved
vulnerabilities/pipelines/v2_importers/collect_repo_fix_commits.py
Outdated
Show resolved
Hide resolved
vulnerabilities/pipelines/v2_importers/collect_repo_fix_commits.py
Outdated
Show resolved
Hide resolved
vulnerabilities/pipelines/v2_importers/collect_repo_fix_commits.py
Outdated
Show resolved
Hide resolved
5ffdf86 to
12dc381
Compare
keshav-space
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ziadhany thanks, see some feedback below.
vulnerabilities/tests/pipelines/v2_importers/test_collect_fix_commit.py
Outdated
Show resolved
Hide resolved
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Add a test for CollectRepoFixCommitPipeline Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
…ct fixed_by_commit_patches correctly. Signed-off-by: ziad hany <ziadhany2016@gmail.com>
… multiple repositories Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Remove CollectOpensslFixCommits Pipeline Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
0b414a9 to
81933a5
Compare
Signed-off-by: ziad hany <ziadhany2016@gmail.com>
keshav-space
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ziadhany, LGTM! Do we have an estimate of how long it takes to run CollectLinuxFixCommitsPipeline?
The Linux kernel pipeline takes around 5 minutes if the network call is ignored. |
I created an initial script to parse Git commit messages that can be easily integrated with our model. The script takes a Git repository as input, parses all commits, and returns the CVEs along with their corresponding fixed commits.
Issues:
results:
vulnerabilities_packagecommitpatch2.zip