Bug #5261
closedExisting src or dst network is not shown when editing a rule
100%
Description
1) Add a rule with src or dst "Network" and enter some network (e.g. 10.11.12.0/24)and save.
2) Edit the rule
firewall_aliases_edit shows the source/destination address as "any" - it loses the "Network" selection and network address.
Note: This is a problem both before and after https://redmine.pfsense.org/issues/5252 which fixed the case of a rule that has "Single host or alias" and an alias specified.
So I have logged this as a separate bug. I will have a look shortly at what is needed to fix this case.
Updated by Anonymous about 9 years ago
- Status changed from New to Feedback
Corrected src/dst type logic
Updated by Anonymous about 9 years ago
- % Done changed from 0 to 100
Applied in changeset pfsense:aadc597321a9cbb41fb76a5faed256ed8a13f96a.
Updated by Phillip Davis about 9 years ago
Now that makes a problem when the src is LANnet, for example. It considers that a 'network' type - see GitHub comment on commit https://github.com/pfsense/pfsense/commit/aadc597321a9cbb41fb76a5faed256ed8a13f96a
Updated by Phillip Davis about 9 years ago
Applied in changeset pfsense:3e8bac5faa97857426e5e7f526e77f484729c26a.
Updated by Phillip Davis about 9 years ago
Can someone else please give this a good testing, because I have done that last change myself so I will easily miss some unusual combination when testing. Things that need to be tested are creating new and editing existing rules with source and/or destination IP like:
a) any
b) special names like LAN net, LAN address, WAN net, WAN address, OPT1 net, OPT1 address, PPPoE clients, L2TP clients and...
c) Single host or alias - put a single IP address and/or an alias name
d) Network - specify a network address with CIDR bit count
Make sure that the current values really are the ones displayed and saved when editing.
It is important that this works correctly because combinations of these are used in every rule, and the rules are the heart of a firewall.
Updated by Chris Buechler almost 9 years ago
- Status changed from Feedback to Resolved
all those circumstances seem fine, been through a variety in the past couple days with no issues found.