Bug 10505 - Alias handling needs improvement
Summary: Alias handling needs improvement
Status: CLOSED FIXED
Alias: None
Product: IPFire
Classification: Unclassified
Component: --- (show other bugs)
Version: 2
Hardware: unspecified Unspecified
: - Unknown - - Unknown -
Assignee: Alexander Marx
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 10486
  Show dependency treegraph
 
Reported: 2014-04-06 18:15 UTC by Michael Tremer
Modified: 2014-05-10 17:21 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Tremer 2014-04-06 18:15:11 UTC
When using aliases with the new firewall, there are multiple issues:

a) The conversion was broken because invalid values have been used for the default alias (= ALL).

http://git.ipfire.org/?p=ipfire-2.x.git;a=commitdiff;h=1d9c1c3079922f6f15354e7001fb2b09ea0355e4

b) Therefore no rule was created for the default alias, which is very unfortunate. The rest of the rules have not been created because the NAT IP address was not found for any alias.

http://git.ipfire.org/?p=ipfire-2.x.git;a=commitdiff;h=085a20ec8bc05b58244bf05d9589e9a1ed3a5265

c) This leaves us now with a working configuration, and it even supports the falsely converted rules.

There is a problem with changing the name of an alias though. As the name is used to identify an alias, rules break when it is changed. It is also possible to create aliases without giving them a name which leads to a similar problem.

So we need a solution for this. Currently, we fall back to the default RED IP address which is total bullocks. A better solution would be to create no rule at all, but the user won't know that this is happening.

Hence I suggest that we convert the names of all aliases in the firewall configuration when the name of an alias is changed. If we cannot do that (for what ever reason), we should fall back to mark the rule in yellow like we do this for no longer existing OpenVPN connections.

We should also make the name for an alias mandatory.
Comment 1 Alexander Marx 2014-04-07 12:06:46 UTC
Should be fixed with this commit:

http://git.ipfire.org/?p=people/amarx/ipfire-2.x.git;a=commit;h=8eabfdafe660212321d585d606e018d4cb8db5f8