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

Severity "lost" when importing a Qualys Scan #10206

Closed
3 tasks
nv-pipo opened this issue May 14, 2024 · 2 comments
Closed
3 tasks

Severity "lost" when importing a Qualys Scan #10206

nv-pipo opened this issue May 14, 2024 · 2 comments
Labels

Comments

@nv-pipo
Copy link
Contributor

nv-pipo commented May 14, 2024

Bug description
When trying to import import a "Qualys Scan" (Qualys VMDR) the severity is lost because the mapping (dict) is expecting an int. Xml's parser returns a string (findtext), so it needs to be converted to int.

Please approve PR: #10205

Basically, it's just a cast from string to int.

Steps to reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Deployment method (select with an X)

  • Docker Compose
  • Kubernetes
  • GoDojo

Environment information

  • Operating System: [e.g. Ubuntu 18.04]
  • DefectDojo version (see footer) or commit message: [use git show -s --format="[%ci] %h: %s [%d]"]

Logs
Use docker-compose logs (or similar, depending on your deployment method) to get the logs and add the relevant sections here showing the error occurring (if applicable).

Sample scan files
If applicable, add sample scan files to help reproduce your problem.

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context (optional)
Add any other context about the problem here.

@nv-pipo nv-pipo added the bug label May 14, 2024
@manuel-sommer
Copy link
Contributor

Can you close this @nv-pipo ?

@nv-pipo
Copy link
Contributor Author

nv-pipo commented Jun 3, 2024

@manuel-sommer
PR #10205 merged. Closing

@nv-pipo nv-pipo closed this as completed Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants