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

Static Route issues in 22.1.10 #5887

Closed
2 tasks done
infinisource opened this issue Jul 18, 2022 · 4 comments
Closed
2 tasks done

Static Route issues in 22.1.10 #5887

infinisource opened this issue Jul 18, 2022 · 4 comments
Assignees
Labels
bug Production bug
Milestone

Comments

@infinisource
Copy link

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Describe the bug

A clear and concise description of what the bug is, including last known working version (if any).

Disabling a static route in the GUI on version 22.1.10 does not withdraw the route from the routing table. Nor does disabling the gateway. Traffic still flows through to the gateway unless the route is deleted. This worked in version 19.7.

To Reproduce

Steps to reproduce the behavior:

  1. Go to System -> Routes -> Configuration
  2. Click on the disabled button for any given static route. IE 10.96.69.0/24
  3. Click apply
  4. Validate that the route is removed via CLI 'netstat -rn | grep 10.96.69.0
  5. In 22.1.10, netstat output would still show the route in the routing table. In 19.7, that route would disappear.

Expected behavior

A clear and concise description of what you expected to happen.

I expect the system to remove a route from the routing table when a route is marked as disabled in the GUI. This clearly worked in version 19.7 but is now broken in the latest version 22.1.10. See the screenshots below for proof.

Describe alternatives you considered

A clear and concise description of any alternative solutions or workaround you considered.

Screenshots

Version 19.7 (WORKING)

opNsense-v19 7_route_add_cli
The screenshot above illustrates on version 19.7 that a static route to 10.96.69.0/24 is active using 'netstat -rn | grep 10.96.'

opNsense-v19 7_route_gui_enabled

The screentshot above shows the same route on 19.7 as being active.

opNsense-v19 7_route_gui_disabled

The screenshot above shows the same route on 19.7 as being disabled.

opNsense-v19 7_route_removal_cli

The screenshot above shows the cli output on 19.7 after disabling the route in the GUI. This clearly works on 19.7.

22.1.10 Version below (NOT WORKING):

opNsense-v22 1 10_route_gui_enabled

The screenshot above shows a route to 10.96.69.0/24 as being enabled.

opNsense-v22 1 10_route_add_cli

The screenshot above shows the route in the routing table via the CLI on 22.1.10

opNsense-v22 1 10_route_gui_disabled

The above screenshot shows the same gui on 22.1.10 but this time the route has been marked as disabled.

opNsense-v22 1 10_route_removal_cli

The above screenshot shows the output of 'netstat -rn | grep 10.96.' on version 22.1.10. The route is clearly still active.

Environment

Version 22.1.10 - Physical Appliance: DEC3860

Version 19.7 - Virtual Machine on ESXi 7.0U3

Additional Info
I found the follow other issues that describe this issue as well:

@AdSchellevis
Copy link
Member

@infinisource thanks for reporting, should be fixed with 4e9baba .

To test locally, execute the following on a console:

opnsense-patch  4e9baba

@infinisource
Copy link
Author

@AdSchellevis - thanks for the hotfix. What's interesting is on my test firewall that is currently on 22.1.10, the patch resolved the issue but on the physical appliance (DEC3860) which is also on 22.1.10, the patch did not resolve the issue.

@infinisource
Copy link
Author

Ok that's odd. Applied the patch twice and the 3rd time it works.

fichtner pushed a commit that referenced this issue Jul 19, 2022
closes #5887

looks like a regression in 52d3e7c, after which delete doesn't seem to execute a valid route command.

(cherry picked from commit 4e9baba)
fichtner pushed a commit that referenced this issue Jul 19, 2022
closes #5887

looks like a regression in 52d3e7c, after which delete doesn't seem to execute a valid route command.

(cherry picked from commit 4e9baba)
@fichtner fichtner added the bug Production bug label Jul 19, 2022
@fichtner fichtner added this to the 22.7 milestone Jul 19, 2022
@AdSchellevis
Copy link
Member

@infinisource ok, weird, well, glad it works now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Production bug
Development

No branches or pull requests

3 participants