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

[Rust] Circular error unable to find scan because unable to find scan #1747

Closed
mryellow opened this issue Oct 29, 2024 · 4 comments
Closed
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@mryellow
Copy link

mryellow commented Oct 29, 2024

Expected behavior

Return an error on errors.

Actual behavior

2024-10-29T00:49:08.027125Z  WARN openvasd::scheduling: unable to fetch results scan_id=dea07b91-461f-46d7-aa14-f29b7be9185d e=Unexpected issue: InvalidResponse(Status { text: "Failed to find scan 'dea07b91-461f-46d7-aa14-f29b7be9185d'", code: StringU32(404) })

Steps to reproduce

Will attempt to find the exact oid which causes the invalid "404".

By including oid one-by-one until it fails, as logs do not indicate what is currently being processed.

@mryellow mryellow added the bug Something isn't working label Oct 29, 2024
@nichtsfrei
Copy link
Member

Is that a duplicate of #1744?

@mryellow
Copy link
Author

Looking closer, it could be.

It seems to indicate some sub-component returned an error with a 404, which then the wrapper reported as a warning with a text from that inner error.

The "repeating over and over" seems to be some kind of internal retry. Which keeps on reporting a 404, where the majority of #1744 isn't that loop, but the initial XML parsing failure and subsequent loss of the socket.

@nichtsfrei
Copy link
Member

That looping behavior will be fixed as well when the scan status is changed to fail, as it removes the scan from the checked background task.

@nichtsfrei nichtsfrei added the duplicate This issue or pull request already exists label Nov 7, 2024
@nichtsfrei
Copy link
Member

This should be fixed with the fixes for #1744.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants