DefectDojo
Add parser and importer for Vulners.com reports API
This pull request has conflicts, please resolve those before we can evaluate the pull request.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Conflicts have been resolved. A maintainer will review the pull request shortly.
@vankyver :
Made a review of the official SDK and it support proxies.
https://github.com/vulnersCom/api/blob/f56dab4ffcdb400da58b680d7feb89cae3550984/vulners/api.py#L97-L105
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
@vankyver :
Made a review of the official SDK and it support proxies.
https://github.com/vulnersCom/api/blob/f56dab4ffcdb400da58b680d7feb89cae3550984/vulners/api.py#L97-L105
@damiencarol I've added fixes with including Vulners SDK
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
@damiencarol Wanted to give you a chance to look at this before it gets merged since you had comments above.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
@vankyver will review this WE. saw a couple of things to slightly modify
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
As @vankyver did not reply over a month, would it be an option to make another PR including these changes plus the requested changes from @damiencarol and close this PR?
@manuel-sommer , @Maffooch : if @vankyver is ok, I can "finish" the PR.
@damiencarol either way works! This PR has two approvals so you could change and merge at your timeline
@damiencarol Was looking at this and was about to merge since it has 2 approvals but couldn't tell from the thread if there was work you still wanted to do.
So, I'll wait to give you a chance to chime it or address the changes you requested. :+1:
@damiencarol From my point of view, everything is ready, so I hope that now we can complete the pull request.