Skip to content

Update GitHub GraphQL import in vulntotal datasources - #1445

Merged
keshav-space merged 3 commits into
aboutcode-org:mainfrom
poju3185:fix-vulntotal-import
May 13, 2024
Merged

Update GitHub GraphQL import in vulntotal datasources#1445
keshav-space merged 3 commits into
aboutcode-org:mainfrom
poju3185:fix-vulntotal-import

Conversation

@poju3185

Copy link
Copy Markdown
Contributor

Fix #1443

Change the usage of vulnerabilities.utils.fetch_github_graphql_query to fetchcode.package_versions.github_response in github.py

@keshav-space

Copy link
Copy Markdown
Member

@poju3185 please add signoff to your commit, and you may want to go through this https://aboutcode.readthedocs.io/en/latest/contributing/writing_good_commit_messages.html.

Change the method used to fetch GitHub GraphQL queries in github.py
Shift from vulnerabilities.utils.fetch_github_graphql_query to fetchcode.package_versions.github_response
The change avoids dependencies on Django apps in vulnerabilities.utils
Signed-off-by: PoJuDeSu <spes9850401@gmail.com>
@poju3185
poju3185 force-pushed the fix-vulntotal-import branch from d809940 to 23d6702 Compare March 21, 2024 15:33
@poju3185

Copy link
Copy Markdown
Contributor Author

I've updated my commit with the sign-off. Thanks for the reminder!

@keshav-space keshav-space added VulnTotal Tool for cross-validating vulnerability labels May 13, 2024
@keshav-space keshav-space changed the title Fix #1443 Update import in vulntotal datasources Update GitHub GraphQL import in vulntotal datasources May 13, 2024
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>

@keshav-space keshav-space left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @poju3185, merging!

@keshav-space
keshav-space merged commit 045fedd into aboutcode-org:main May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

VulnTotal Tool for cross-validating vulnerability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AppRegistryNotReady error when setting up vulntotal

2 participants