Page MenuHomePhabricator

use /etc/network/if-pre-up.d/ instead of /etc/network/interfaces to load Whonix's firewall
Closed, ResolvedPublic

Description

Debian bug #700811: interface comes up even if a script in /etc/network/if-pre-up.d/ fails has been fixed.

It would be better to drop Whonix-Workstation's (optional) and Whonix-Gateway's firewall load hook in /etc/network/if-pre-up.d/ than doing what we're doing right now (using pre-up in /etc/network/interfaces). This would make /etc/network/interfaces cleaner, easier to port and more difficult for users to remove pre-up from /etc/network/interfaces and shoot their own feet.

We have to wait until Debian Jessie gets stable, because this bug ix not fixed in Debian Wheezy.

Details

Impact
Needs Triage

Event Timeline

JasonJAyalaP raised the priority of this task from to Normal.
JasonJAyalaP updated the task description. (Show Details)
Patrick set Impact to Needs Triage.
Patrick added subscribers: nrgaway, HulaHoop.
Patrick changed the task status from Open to Review.May 23 2015, 1:50 PM

Removed 'pre-up /usr/bin/whonix_firewall', because /etc/network/if-pre-up.d to load the firewall, because of a Debian upstream bug interface comes up even if a script in /etc/network/if-pre-up.d/ fails http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700811 was fixed. - https://phabricator.whonix.org/T68:

Made package more standalone. Requiring 'pre-up /usr/bin/whonix_firewall' in /etc/network/interfaces is no longer necessary. Added etc/network/if-pre-up.d/30_whonix_firewall to load the firewall, because of a Debian upstream bug 'interface comes up even if a script in /etc/network/if-pre-up.d/ fails' http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=700811 was fixed. - https://phabricator.whonix.org/T68:

In T68#4883, @HulaHoop wrote:

There is currently no problem that needs to be solved.

Done, tested and functional in Whonix 11.0.0.2.0-developers-only.