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

Collected and added advisories from zerodayinitiative #1818

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

Conversation

Dedsec0098
Copy link

Fix #1471

Changes made :

  1. Added ZDI importer to import advisories (fetching, parsing, importing)

  2. Added test file for the importer

  3. Added Test data files

@pombredanne I have added the pipeline to import the advisories and If you have any suggestions please let me know I will make the changes required.

@Dedsec0098
Copy link
Author

@pombredanne I have already added pipeline_id = "zdi_importer" but still I am getting the error when I test the changes locally.

______________________________________ PipelineTests.test_no_none_pipeline_ids ______________________________________

self = <vulnerabilities.tests.pipelines.test_pipeline_id.PipelineTests testMethod=test_no_none_pipeline_ids>

    def test_no_none_pipeline_ids(self):
        none_pipeline_ids = [cls for cls, pid in self.pipeline_dict.items() if pid == None]
    
        if none_pipeline_ids:
            error_messages = [f"{cls.__name__} has None pipeline_id." for cls in none_pipeline_ids]
            error_message = "`pipeline_id` should not be None:\n" + "\n".join(error_messages)
>           assert False, error_message
E           AssertionError: `pipeline_id` should not be None:
E             VulnerableCodeBaseImporterPipeline has None pipeline_id.
E           assert False

vulnerabilities/tests/pipelines/test_pipeline_id.py:61: AssertionError

Can you please suggest what should I do in this case.

@kunalsz
Copy link

kunalsz commented Mar 20, 2025

@Dedsec0098 you didn't add the license url for the pipeline

@keshav-space keshav-space added the vibe-code Completely AI-generated code label Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vibe-code Completely AI-generated code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collect data from https://www.zerodayinitiative.com/advisories/
3 participants