Misleading error message for plugin version check

[leogr]

Motivation

Currently, when a plugin returns an empty string for its version or for the required API version, the user will get the following error message:

Could not parse version string from

In Falco, the full error line will look like this:

Tue Mar  8 17:26:15 2022: Runtime error: cannot load plugin /path/to/libsomething.so: Could not parse version string from . Exiting.

Note that my plugin returned an empty string for the required API version in those examples.

Feature

• The error message should report if the error refers to the plugin version or the required API version
• The empty string case should be rendered in a clear way (or a different error message should be returned)

Alternatives

Doing nothing and risk a lot of headaches :smile_cat:

Additional context

Relevant code parts: https://github.com/falcosecurity/libs/blob/84470f1621d143c8cdc24fdf9bb4c5b87180ea6c/userspace/libsinsp/plugin.cpp#L366-L370

https://github.com/falcosecurity/libs/blob/84470f1621d143c8cdc24fdf9bb4c5b87180ea6c/userspace/libsinsp/plugin.cpp#L673-L677

[poiana]

Issues go stale after 90d of inactivity.

Mark the issue as fresh with /remove-lifecycle stale.

Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Provide feedback via https://github.com/falcosecurity/community.

/lifecycle stale