[gnome-control-center] network: Remove support for firewalld zones
- From: Bastien Nocera <hadess src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-control-center] network: Remove support for firewalld zones
- Date: Thu, 13 Apr 2017 08:25:08 +0000 (UTC)
commit b36924567e16839fbdc8796b975970e214719d81
Author: Bastien Nocera <hadess hadess net>
Date: Wed Apr 12 18:19:24 2017 +0200
network: Remove support for firewalld zones
Firewalld is a Fedora-only daemon, and it doesn't integrate all that
well with Fedora Workstation, the version of Fedora that uses GNOME, so
remove the support.
In Fedora Workstation, firewalld only ever supports one zone, as the
other ones are badly defined, and not translatable or translated.
panels/network/connection-editor/Makefile.am | 2 -
.../network/connection-editor/ce-page-ethernet.c | 10 --
.../network/connection-editor/ce-page-security.c | 14 --
.../network/connection-editor/ce-page-security.h | 2 -
panels/network/connection-editor/ce-page-vpn.c | 11 --
panels/network/connection-editor/ethernet-page.ui | 30 -----
.../network/connection-editor/firewall-helpers.c | 125 --------------------
.../network/connection-editor/firewall-helpers.h | 33 -----
panels/network/connection-editor/security-page.ui | 32 -----
panels/network/connection-editor/vpn-page.ui | 39 ------
10 files changed, 0 insertions(+), 298 deletions(-)
---
diff --git a/panels/network/connection-editor/Makefile.am b/panels/network/connection-editor/Makefile.am
index de8039f..fe74aa7 100644
--- a/panels/network/connection-editor/Makefile.am
+++ b/panels/network/connection-editor/Makefile.am
@@ -30,8 +30,6 @@ libconnection_editor_la_SOURCES = \
ce-page-vpn.c \
vpn-helpers.h \
vpn-helpers.c \
- firewall-helpers.h \
- firewall-helpers.c \
ui-helpers.h \
ui-helpers.c
diff --git a/panels/network/connection-editor/ce-page-ethernet.c
b/panels/network/connection-editor/ce-page-ethernet.c
index 75ee324..69866f6 100644
--- a/panels/network/connection-editor/ce-page-ethernet.c
+++ b/panels/network/connection-editor/ce-page-ethernet.c
@@ -28,7 +28,6 @@
#include <NetworkManager.h>
-#include "firewall-helpers.h"
#include "ce-page-ethernet.h"
#include "ui-helpers.h"
@@ -66,7 +65,6 @@ connect_ethernet_page (CEPageEthernet *page)
char **mac_list;
const char *s_mac_str;
GtkWidget *widget;
- GtkWidget *heading;
const gchar *name;
const gchar *cloned_mac;
@@ -114,11 +112,6 @@ connect_ethernet_page (CEPageEthernet *page)
g_signal_connect (widget, "toggled",
G_CALLBACK (all_user_changed), page);
g_signal_connect_swapped (widget, "toggled", G_CALLBACK (ce_page_changed), page);
-
- widget = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder, "combo_zone"));
- heading = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder, "heading_zone"));
- firewall_ui_setup (sc, widget, heading, CE_PAGE (page)->cancellable);
- g_signal_connect_swapped (widget, "changed", G_CALLBACK (ce_page_changed), page);
}
static void
@@ -147,9 +140,6 @@ ui_to_setting (CEPageEthernet *page)
NM_SETTING_CONNECTION_ID, gtk_entry_get_text (page->name),
NULL);
- entry = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder, "combo_zone"));
- firewall_ui_to_setting (page->setting_connection, entry);
-
g_free (cloned_mac);
g_free (device_mac);
}
diff --git a/panels/network/connection-editor/ce-page-security.c
b/panels/network/connection-editor/ce-page-security.c
index 9789c75..1f0eceb 100644
--- a/panels/network/connection-editor/ce-page-security.c
+++ b/panels/network/connection-editor/ce-page-security.c
@@ -28,7 +28,6 @@
#include "wireless-security.h"
#include "ce-page-security.h"
-#include "firewall-helpers.h"
G_DEFINE_TYPE (CEPageSecurity, ce_page_security, CE_TYPE_PAGE)
@@ -148,7 +147,6 @@ security_combo_changed (GtkComboBox *combo,
gtk_container_remove (GTK_CONTAINER (parent), sec_widget);
gtk_size_group_add_widget (page->group, page->security_heading);
- gtk_size_group_add_widget (page->group, page->firewall_heading);
wireless_security_add_to_size_group (sec, page->group);
gtk_container_add (GTK_CONTAINER (vbox), sec_widget);
@@ -205,7 +203,6 @@ finish_setup (CEPageSecurity *page)
NMConnection *connection = CE_PAGE (page)->connection;
NMSettingWireless *sw;
NMSettingWirelessSecurity *sws;
- NMSettingConnection *sc;
gboolean is_adhoc = FALSE;
GtkListStore *sec_model;
GtkTreeIter iter;
@@ -352,13 +349,6 @@ finish_setup (CEPageSecurity *page)
page->security_combo = combo;
- page->firewall_heading = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder,
"heading_zone"));
- page->firewall_combo = GTK_COMBO_BOX (gtk_builder_get_object (CE_PAGE (page)->builder,
"combo_zone"));
-
- sc = nm_connection_get_setting_connection (CE_PAGE (page)->connection);
- firewall_ui_setup (sc, GTK_WIDGET (page->firewall_combo), page->firewall_heading, CE_PAGE
(page)->cancellable);
- g_signal_connect_swapped (page->firewall_combo, "changed", G_CALLBACK (ce_page_changed), page);
-
security_combo_changed (combo, page);
g_signal_connect (combo, "changed",
G_CALLBACK (security_combo_changed), page);
@@ -370,7 +360,6 @@ validate (CEPage *page,
GError **error)
{
NMSettingWireless *sw;
- NMSettingConnection *sc;
WirelessSecurity *sec;
gboolean valid = FALSE;
const char *mode;
@@ -413,9 +402,6 @@ validate (CEPage *page,
valid = TRUE;
}
- sc = nm_connection_get_setting_connection (connection);
- firewall_ui_to_setting (sc, GTK_WIDGET (CE_PAGE_SECURITY (page)->firewall_combo));
-
return valid;
}
diff --git a/panels/network/connection-editor/ce-page-security.h
b/panels/network/connection-editor/ce-page-security.h
index 6f692a2..fa7e731 100644
--- a/panels/network/connection-editor/ce-page-security.h
+++ b/panels/network/connection-editor/ce-page-security.h
@@ -45,8 +45,6 @@ struct _CEPageSecurity
GtkComboBox *security_combo;
GtkWidget *security_heading;
- GtkComboBox *firewall_combo;
- GtkWidget *firewall_heading;
GtkSizeGroup *group;
gboolean adhoc;
};
diff --git a/panels/network/connection-editor/ce-page-vpn.c b/panels/network/connection-editor/ce-page-vpn.c
index 533aa8b..6305116 100644
--- a/panels/network/connection-editor/ce-page-vpn.c
+++ b/panels/network/connection-editor/ce-page-vpn.c
@@ -28,7 +28,6 @@
#include "ce-page-vpn.h"
#include "vpn-helpers.h"
-#include "firewall-helpers.h"
G_DEFINE_TYPE (CEPageVpn, ce_page_vpn, CE_TYPE_PAGE)
@@ -122,7 +121,6 @@ connect_vpn_page (CEPageVpn *page)
{
const gchar *name;
GtkWidget *widget;
- GtkWidget *heading;
name = nm_setting_connection_get_id (page->setting_connection);
gtk_entry_set_text (page->name, name);
@@ -134,12 +132,6 @@ connect_vpn_page (CEPageVpn *page)
nm_setting_connection_get_num_permissions (page->setting_connection)
== 0);
g_signal_connect (widget, "toggled",
G_CALLBACK (all_user_changed), page);
-
- widget = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder, "combo_zone"));
- heading = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder, "heading_zone"));
- firewall_ui_setup (page->setting_connection, widget, heading, CE_PAGE (page)->cancellable);
- g_signal_connect_swapped (widget, "changed", G_CALLBACK (ce_page_changed), page);
-
}
static gboolean
@@ -148,13 +140,10 @@ validate (CEPage *page,
GError **error)
{
CEPageVpn *self = CE_PAGE_VPN (page);
- GtkWidget *widget;
g_object_set (self->setting_connection,
NM_SETTING_CONNECTION_ID, gtk_entry_get_text (self->name),
NULL);
- widget = GTK_WIDGET (gtk_builder_get_object (CE_PAGE (page)->builder, "combo_zone"));
- firewall_ui_to_setting (self->setting_connection, widget);
if (!nm_setting_verify (NM_SETTING (self->setting_connection), NULL, error))
return FALSE;
diff --git a/panels/network/connection-editor/ethernet-page.ui
b/panels/network/connection-editor/ethernet-page.ui
index 59d1a35..af7564a 100644
--- a/panels/network/connection-editor/ethernet-page.ui
+++ b/panels/network/connection-editor/ethernet-page.ui
@@ -262,36 +262,6 @@
<child>
<placeholder/>
</child>
- <child>
- <object class="GtkLabel" id="heading_zone">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="xalign">1</property>
- <property name="label" translatable="yes">Firewall _Zone</property>
- <property name="use_underline">True</property>
- <property name="mnemonic_widget">combo_zone</property>
- </object>
- <packing>
- <property name="left_attach">0</property>
- <property name="top_attach">4</property>
- <property name="width">1</property>
- <property name="height">1</property>
- </packing>
- </child>
- <child>
- <object class="GtkComboBoxText" id="combo_zone">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="entry_text_column">0</property>
- <property name="id_column">1</property>
- </object>
- <packing>
- <property name="left_attach">1</property>
- <property name="top_attach">4</property>
- <property name="width">1</property>
- <property name="height">1</property>
- </packing>
- </child>
</object>
<object class="GtkSizeGroup" id="sizegroup1">
<widgets>
diff --git a/panels/network/connection-editor/firewall-helpers.c
b/panels/network/connection-editor/firewall-helpers.c
index 2e2c29f..e69de29 100644
--- a/panels/network/connection-editor/firewall-helpers.c
+++ b/panels/network/connection-editor/firewall-helpers.c
@@ -1,125 +0,0 @@
-/* -*- Mode: C; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 8 -*-
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * (C) Copyright 2013 Red Hat, Inc.
- */
-
-#include "config.h"
-
-#include <glib/gi18n.h>
-#include <gio/gio.h>
-
-#include "firewall-helpers.h"
-
-typedef struct {
- gchar *zone;
- GtkWidget *combo;
- GtkWidget *label;
-} GetZonesReplyData;
-
-static void
-get_zones_reply (GObject *source,
- GAsyncResult *res,
- gpointer user_data)
-{
- GDBusConnection *bus = G_DBUS_CONNECTION (source);
- GetZonesReplyData *d = user_data;
- GVariant *ret;
- GError *error = NULL;
- const gchar **zones;
- gint idx;
- gint i;
-
- ret = g_dbus_connection_call_finish (bus, res, &error);
-
- gtk_combo_box_text_append_text (GTK_COMBO_BOX_TEXT (d->combo),
- C_("Firewall zone", "Default"));
- gtk_widget_set_tooltip_text (d->combo, _("The zone defines the trust level of the connection"));
-
- idx = 0;
- if (error) {
- gtk_widget_hide (d->combo);
- gtk_widget_hide (d->label);
- g_error_free (error);
- }
- else {
- gtk_widget_show (d->combo);
- gtk_widget_show (d->label);
- g_variant_get (ret, "(^a&s)", &zones);
-
- for (i = 0; zones[i]; i++) {
- gtk_combo_box_text_append_text (GTK_COMBO_BOX_TEXT (d->combo),
- zones[i]);
- if (g_strcmp0 (d->zone, zones[i]) == 0)
- idx = i + 1;
- }
- if (d->zone && idx == 0) {
- gtk_combo_box_text_append_text (GTK_COMBO_BOX_TEXT (d->combo),
- d->zone);
- idx = i + 1;
- }
- g_variant_unref (ret);
- }
- gtk_combo_box_set_active (GTK_COMBO_BOX (d->combo), idx);
-
- g_free (d->zone);
- g_free (d);
-}
-
-void
-firewall_ui_setup (NMSettingConnection *setting,
- GtkWidget *combo,
- GtkWidget *label,
- GCancellable *cancellable)
-{
- GDBusConnection *bus;
- GetZonesReplyData *d;
-
- bus = g_bus_get_sync (G_BUS_TYPE_SYSTEM, NULL, NULL);
-
- d = g_new0 (GetZonesReplyData, 1);
- d->zone = g_strdup (nm_setting_connection_get_zone (setting));
- d->combo = combo;
- d->label = label;
-
- g_dbus_connection_call (bus,
- "org.fedoraproject.FirewallD1",
- "/org/fedoraproject/FirewallD1",
- "org.fedoraproject.FirewallD1.zone",
- "getZones",
- NULL,
- NULL,
- 0,
- G_MAXINT,
- cancellable,
- get_zones_reply, d);
- g_object_unref (bus);
-}
-
-void
-firewall_ui_to_setting (NMSettingConnection *setting, GtkWidget *combo)
-{
- gchar *zone;
-
- zone = gtk_combo_box_text_get_active_text (GTK_COMBO_BOX_TEXT (combo));
- if (g_strcmp0 (zone, C_("Firewall zone", "Default")) == 0) {
- g_free (zone);
- zone = NULL;
- }
-
- g_object_set (setting, NM_SETTING_CONNECTION_ZONE, zone, NULL);
- g_free (zone);
-}
diff --git a/panels/network/connection-editor/firewall-helpers.h
b/panels/network/connection-editor/firewall-helpers.h
index 213c515..e69de29 100644
--- a/panels/network/connection-editor/firewall-helpers.h
+++ b/panels/network/connection-editor/firewall-helpers.h
@@ -1,33 +0,0 @@
-/*
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * (C) Copyright 2013 Red Hat, Inc.
- */
-
-#ifndef _FIREWALL_HELPERS_H_
-#define _FIREWALL_HELPERS_H_
-
-#include <NetworkManager.h>
-#include <gtk/gtk.h>
-
-void firewall_ui_setup (NMSettingConnection *setting,
- GtkWidget *combo,
- GtkWidget *label,
- GCancellable *cancellable);
-void firewall_ui_to_setting (NMSettingConnection *setting,
- GtkWidget *combo);
-
-
-#endif /* _FIREWALL_HELPERS_H_ */
diff --git a/panels/network/connection-editor/security-page.ui
b/panels/network/connection-editor/security-page.ui
index d5c1720..0c3dd92 100644
--- a/panels/network/connection-editor/security-page.ui
+++ b/panels/network/connection-editor/security-page.ui
@@ -63,38 +63,6 @@
<property name="height">1</property>
</packing>
</child>
- <child>
- <object class="GtkLabel" id="heading_zone">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="xalign">1</property>
- <property name="label" translatable="yes">Firewall _Zone</property>
- <property name="use_underline">True</property>
- <property name="mnemonic_widget">combo_zone</property>
- </object>
- <packing>
- <property name="left_attach">0</property>
- <property name="top_attach">2</property>
- <property name="width">1</property>
- <property name="height">1</property>
- </packing>
- </child>
- <child>
- <object class="GtkComboBoxText" id="combo_zone">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="hexpand">True</property>
- <property name="entry_text_column">0</property>
- <property name="id_column">1</property>
- </object>
- <packing>
- <property name="left_attach">1</property>
- <property name="top_attach">2</property>
- <property name="width">1</property>
- <property name="height">1</property>
- </packing>
- </child>
-
</object>
</child>
<child type="tab">
diff --git a/panels/network/connection-editor/vpn-page.ui b/panels/network/connection-editor/vpn-page.ui
index dea9a4a..5798fd1 100644
--- a/panels/network/connection-editor/vpn-page.ui
+++ b/panels/network/connection-editor/vpn-page.ui
@@ -51,45 +51,6 @@
</packing>
</child>
<child>
- <object class="GtkBox" id="box1">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="spacing">6</property>
- <child>
- <object class="GtkLabel" id="heading_zone">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="label" translatable="yes">Firewall _Zone</property>
- <property name="use_underline">True</property>
- <property name="mnemonic_widget">combo_zone</property>
- </object>
- <packing>
- <property name="expand">False</property>
- <property name="fill">True</property>
- <property name="position">0</property>
- </packing>
- </child>
- <child>
- <object class="GtkComboBoxText" id="combo_zone">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="entry_text_column">0</property>
- <property name="id_column">1</property>
- </object>
- <packing>
- <property name="expand">True</property>
- <property name="fill">True</property>
- <property name="position">1</property>
- </packing>
- </child>
- </object>
- <packing>
- <property name="expand">False</property>
- <property name="fill">True</property>
- <property name="position">1</property>
- </packing>
- </child>
- <child>
<object class="GtkCheckButton" id="all_user_check">
<property name="label" translatable="yes">Make available to other _users</property>
<property name="visible">True</property>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]