feature: ui: added progress bar for vulnerabilities.importers

[harsh098]

In response to Issue #1376 . I am pushing the changes that add a progress bar to the vulnerabilitites.importers.nvd.NVDImporter

• Please Review and suggest changes/improvements.
• I'll replicate these changes to all other importers.

Dependencies Added

• progress==1.6 allows addition of progress bars to tty consoles.

Screenshots

[TG1999]

@harsh098 thanks! you may need to re-write your commit message, your commit message needs to be imperative, have a look on this https://cbea.ms/git-commit/

[harsh098]

I have changed commit message please check @TG1999 also shall i propagate this change to all other importers ?

[TG1999]

@harsh098 the commit message should be Add progress bar to NVDImporter and not Added progress bar to NVDImporter, commit messages should be imperative like a command given to the codebase.

also shall i propagate this change to all other importers ?

Please do!

[harsh098]

Progress bar has been added to following Importers

• nvd.NVDImporter
• github.GitHubAPIImporter
• gitlab.GitLabAPIImporter
• npm.NpmImporter
• pypa.PyPaImporter
• nginx.NginxImporter
• pysec.PyPIImporter

Working on rest of importers

[harsh098]

An important thing to consider,
The recent workflows failed as dependency
progress==1.6
was added to requirements.txt in this commit 8cee434 was not installed by the workflow

@TG1999 and @ziadhany please suggest requisite changes from my end

[harsh098]

@TG1999 and @ziadhany
I forgot to add the dependency progress==1.6 to setup.cfg.
Please rerun the checks.
The lastest commit 7f5eb72 fixes this.
I apologise for the carelessness

[ziadhany]

@TG1999 and @ziadhany I forgot to add the dependency progress==1.6 to setup.cfg. Please rerun the checks. The lastest commit 7f5eb72 fixes this. I apologise for the carelessness

No worries about carelessness! We all need a helping hand sometimes

[harsh098]

Sure @ziadhany ,
As discussed above this PR addresses the issue #1376 ,
Are there any further additions required else I'm ma' movin' to other issues I'm researching on to work ?

[ziadhany]

@harsh098 thanks for your effort , final nits for your consideration

[keshav-space]

In the future iteration, it would be much better to abstract the progress logging within the ImportRunner().run() itself so that we don't need to individually log the progress inside each importer.

[harsh098]

Made the requested changes please have a look @ziadhany and @TG1999