-
-
Notifications
You must be signed in to change notification settings - Fork 17.9k
Labels
0.kind: bugSomething is brokenSomething is broken9.needs: module (update)This needs a module to be changedThis needs a module to be changed9.needs: package (update)This needs a package to be updatedThis needs a package to be updated9.needs: testsThis PR needs testsThis PR needs tests9.needs: upstream fixThis PR needs upstream to change somethingThis PR needs upstream to change something
Description
Currently, opensnitch is very broken in various ways.
Fixing it is non-trivial, and needs a bit more tracking than just a PR. And, ideally, documentation about what is broken and the fixes in progress.
Broken:
opensnitch-ebpfkernel module is missing passthru testsopensnitch-ebpfkernel module maintainers are not the same as for opensnitchopensnitchnixos module has inconsistent maintainer listopensnitchnixos VM test does not check the backend actually being used is the same as defined in config, meaning even whenprocfallback takes effect the test passesopensnitchnixos VM test does not check for successful ebpf module loadingopensnitchnixos VM test does not check for presence of audit rulesopensnitch-uidoes not correctly display information on 4 digit version string: opensnitch: 4 digit version breaks opensnitch-ui connectionย #418773opensnitchdsystemd service is missingauditpackage in its$PATHwhen monitoring method isauditauditmonitoring method relies onaudisp-af_unixplugin inauditdauditddoes not support plugins on nixos yet- making
auditdsupport plugins either needs a patch (ideally upstream) to allow symlinks for the plugin config file: https://github.com/linux-audit/audit-userspace/blob/3ab32ecd9924ed0cd4f41a05ff91594b30fe0a25/audisp/audispd.c#L121-L122
There might be more problems, these are the problems i have identified so far.
cc @onny
ddogfoodd
Metadata
Metadata
Assignees
Labels
0.kind: bugSomething is brokenSomething is broken9.needs: module (update)This needs a module to be changedThis needs a module to be changed9.needs: package (update)This needs a package to be updatedThis needs a package to be updated9.needs: testsThis PR needs testsThis PR needs tests9.needs: upstream fixThis PR needs upstream to change somethingThis PR needs upstream to change something