diff options
author | Mathieu Bridon <bochecha@daitauha.fr> | 2018-01-09 20:39:45 +0100 |
---|---|---|
committer | fiaxh <fiaxh@users.noreply.github.com> | 2018-01-09 20:39:45 +0100 |
commit | 5557c03be87a622fa830b26e27ca6b158093937c (patch) | |
tree | 6540ebe204e58e2196ba4ba75284dc38d241ebe2 /libdino/src/dbus/network_manager_dbus_properties.vala | |
parent | 5d6cf9d8d56f9f3f781d1e6afa149f736868a108 (diff) | |
download | dino-5557c03be87a622fa830b26e27ca6b158093937c.tar.gz dino-5557c03be87a622fa830b26e27ca6b158093937c.zip |
Move to GNetworkMonitor (#236)
* Move to GNetworkMonitor
Dino currently talks to NetworkManager over DBus to know the state of
the network.
That doesn't work in a Flatpak sandbox by default though, because
Flatpak filters DBus communications and only allows a very small set of
things to pass (which are known to be safe).
Gio provides an API to know the state of the network (and be notified of
changes via a signal): GNetworkMonitor.
And GNetworkMonitor works both inside a Flatpak sandbox, and in
traditional builds. (in Flatpak it uses what we call a "portal", which
are the clean, safe way to let apps exit their sandbox)
Fixes #235
* Don't check for network connectivity for now
The connectivity check really is the correct thing to do:
* network_available means that the computer has network routes to
"somewhere". That is, it is connected to a router.
* connectivity.FULL means that the computer can access "the
Internet". That is, if it is behind a router, that router is
connected.
As a result, only checking for network_available is not correct.
Unfortunately, NetworkManager tends to wait a long time before checking
for connectivity. As a result, it is possible that a transient network
error leaves NetworkManager thinking that network_available is true but
connectivity!=FULL, and it will wait several minutes before realizing
that the Internet connexion did come back.
During that time, apps checking for connectivity (e.g the whole GNOME
desktop) will think they don't have access to the Internet, while apps
that don't (e.g Firefox) will access the Internet just fine. Users are
understandably confused when that happens.
Removing the check for connectivity is an acceptable trade-off in the
short-term, until this situation is improved on the NetworkManager side.
https://bugzilla.gnome.org/show_bug.cgi?id=792240
Diffstat (limited to 'libdino/src/dbus/network_manager_dbus_properties.vala')
-rw-r--r-- | libdino/src/dbus/network_manager_dbus_properties.vala | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/libdino/src/dbus/network_manager_dbus_properties.vala b/libdino/src/dbus/network_manager_dbus_properties.vala deleted file mode 100644 index 37cf76cb..00000000 --- a/libdino/src/dbus/network_manager_dbus_properties.vala +++ /dev/null @@ -1,14 +0,0 @@ -[DBus (name = "org.freedesktop.DBus.Properties")] -public interface NetworkManagerDBusProperties : GLib.Object { - public signal void properties_changed(string iface, HashTable<string, Variant> changed, string[] invalidated); -} - -public static NetworkManagerDBusProperties? get_dbus_properties() { - NetworkManagerDBusProperties? dbus_properties = null; - try { - dbus_properties = Bus.get_proxy_sync(BusType.SYSTEM, "org.freedesktop.DBus.Properties", "/org/freedesktop/NetworkManager"); - } catch (IOError e) { - stderr.printf("%s\n", e.message); - } - return dbus_properties; -} |