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.

Sign up for GitHub

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

Jump to bottom

[Bug] Missing Rule Still Displayed In UI #1056

Closed
hbednar opened this issue Oct 10, 2023 · 2 comments
Closed

[Bug] Missing Rule Still Displayed In UI #1056

hbednar opened this issue Oct 10, 2023 · 2 comments

Comments

@hbednar
Copy link

hbednar commented Oct 10, 2023

Describe the bug
After doing a review of rules and deleting the ones that are no longer needed, i got the below error when trying to delete one.
As this is an old rule it may have been deleted and for some reason is still showing in the UI.

Include the following information:

  • OpenSnitch version: 1.6.2
    • OpenSnitch UI version: 1.6.3
  • OS: Debian
  • Version sid
  • Window Manager: i3wm
  • Kernel version: echo $(uname -a)

To Reproduce
I found this out by accident when trying to delete a rule but, it may also happen when editing a rule that's missing.

Post error logs:
2023-55
@gustavo-iniguez-goya
Copy link
Collaborator

Hi @hbednar ,

Thank you for reporting this problem. I've managed to reproduce an (old) issue that was preventing rules from being refreshed.
As the list of rules was not refreshed, when you clicked on an already deleted rule, the error you posted was usually displayed to the user.

With this fix this error should happen anymore. But if you apply the fix, or reproduce it again with the next release, please, reopen this issue.

Thank you!

@hbednar
Copy link
Author

hbednar commented Oct 13, 2023

@gustavo-iniguez-goya
Thank you for having a look into this, for info when does opensnitch refreshed the rule list.

gustavo-iniguez-goya added a commit that referenced this issue Nov 6, 2023
@gustavo-iniguez-goya
ui,rules: fixed refrehsing list, fixed typos
47ca758 
- Fixed refreshing list when deleting rules.
- Fixed typos that were preventing errors from being displayed correctly.

Closes: #1056
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gustavo-iniguez-goya @hbednar