Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Misleading error message #769

Open
larsroger opened this issue May 14, 2024 · 3 comments
Open

Misleading error message #769

larsroger opened this issue May 14, 2024 · 3 comments

Comments

@larsroger
Copy link

larsroger commented May 14, 2024

When installing the Snyk-broker on a Openshift-platform with limitations on file access due to security context a misleading error message is given. This will, at least it did for us, possibly result in time waste for the "sys-admins". We spent many ours trying to figure out what the No Filters found. A Broker requires filters to run. Shutting down.-message meant, as there is no reference to any filters in the documentation or in the Helm-chart. We finally had a look at the source code and was able to backtrack the message

Expected behaviour

If the application is not able to load the files in defaultFilters a complementary and precise error message should be given. In our case the process did not have access to load the files (access denied)

Actual behaviour

The error message No Filters found. A Broker requires filters to run. Shutting down. is given.

`No Filters found. A Broker requires filters to run. Shutting down.`,

Steps to reproduce

Make the files in defaultFilters inaccessible for the process while the application starts

@aarlaud
Copy link
Contributor

aarlaud commented May 14, 2024

ACK. We'll try to make this message clearer moving forward.

@larsroger
Copy link
Author

Great 👍 And thanks for the quick reply!

@Khlooddeeb
Copy link

👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

3 participants