Re: [PATCH 00/12] Improve WireGuard disconnect, error and hostname lookup

> On 21 Mar 2025, at 11:31 pm, Jussi Laakkonen <[email protected]> wrote:
> 
> Hi Christian and all,
> 
>> I’ve been using this series combined with the following series and patch for several weeks:
>> https://patchwork.kernel.org/project/connman/cover/[email protected]/
>> https://patchwork.kernel.org/project/connman/patch/DB6PR10MB1845EBB07DB785B580B3F6ECE1E02@DB6PR10MB1845.EURPRD10.PROD.OUTLOOK.COM/
>> Plus a revert of this commit which I’ve previously flagged as breaking the default “route all traffic down tunnel” with WireGuard (the main use-case for LibreELEC users):
>> https://git.kernel.org/pub/scm/network/connman/connman.git/commit/?id=9eb1772d31b6fcf78e8711976696491aec9ff5df
>> For easier testing all patches are in this branch: https://github.com/chewitt/connman/commits/wireguard
>> No issues observed, so:
>> Tested-by: Christian Hewitt <[email protected]>
> 
> I'm happy to hear. I have been fixing other issues related to WG in the past weeks a lot and considering to push all of the changes into one big WG patch set, as this has been left dangling.
> 
> I haven't fully tested this on top of upstream ConnMan, but if you or someone else is interested to see if the issues that were previously encountering are fixed or not.
> 
> All changes are available here, on top of latest upstream ConnMan: https://github.com/LaakkonenJussi/connman_upstream/pull/9
> 
> It is a PR to the internal repository so anyone can comment there too. Decided to put this into public as Mullvad seems to give only IP addresses and cannot really rely on demo.wireguard.com alone on this. So, feel free to test, if it works I'll setup a patch set, maybe combine some of the changes to make fewer patches.
> 
> Have a nice weekend you all!

Thanks for sharing Jussi. Minor compile error fixed with:

diff --git a/src/shared/util.h b/src/shared/util.h
index 44e07b84..9af0303d 100644
--- a/src/shared/util.h
+++ b/src/shared/util.h
@@ -25,7 +25,7 @@
   #include <glib.h>
 #include <stdbool.h>
-#include <inet.h>
+#include <connman/inet.h>
   #define AF_INET_POS 0
 #define AF_INET6_POS 1

An existing WireGuard configuration on the family daily-driver RPi5 running LibreELEC shows no obvious regressions with a private VPN server (not a commercial service). Lots more testing required :)

Christian