[EGIT] [enlightenment] 04/08: networkmanager: raise username diagnostics to WRN so they print

[Enlightenment Git]

This is an automated email from the git hooks/post-receive script.

git pushed a commit to branch nm-vpn-username-capture
in repository enlightenment.

View the commit online.

commit 9c2911b36631b1616179ca0f80d9dd74abdc93ab
Author: cedric.b...@free.fr <cedric.b...@free.fr>
AuthorDate: Sun Jun 21 22:24:38 2026 -0600

    networkmanager: raise username diagnostics to WRN so they print
    
    The module's INF logs go to the 'networkmanager' eina domain which sits at
    the default WARN level, so import->probe->dialog diagnostics were silently
    dropped (only import.c's INF, which resolves to E's core domain, showed).
    Use WRN for the diagnostic trace so it is visible without env tweaks.
    
    Co-Authored-By: Claude Opus 4.8 (1M context) <nore...@anthropic.com>
---
 src/modules/networkmanager/e_mod_main.c                    | 2 +-
 src/modules/networkmanager/e_networkmanager_vpn_username.c | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/modules/networkmanager/e_mod_main.c b/src/modules/networkmanager/e_mod_main.c
index 8d626f4b6..6a6bb0997 100644
--- a/src/modules/networkmanager/e_mod_main.c
+++ b/src/modules/networkmanager/e_mod_main.c
@@ -413,7 +413,7 @@ _enm_vpn_import_done_cb(void *data EINA_UNUSED, Eina_Bool ok, const char *err,
    E_Dialog *err_dlg;
    if (ok)
      {
-        INF("VPN import succeeded");
+        WRN("VPN import succeeded; conn_name(uuid)=%s", conn_name ?: "(null)");
         if (conn_name) enm_vpn_username_maybe_prompt(conn_name);
         return;
      }
diff --git a/src/modules/networkmanager/e_networkmanager_vpn_username.c b/src/modules/networkmanager/e_networkmanager_vpn_username.c
index 4aa8c3bbc..697b265ee 100644
--- a/src/modules/networkmanager/e_networkmanager_vpn_username.c
+++ b/src/modules/networkmanager/e_networkmanager_vpn_username.c
@@ -116,7 +116,7 @@ enm_vpn_username_set(const char *conn_name, const char *username,
             "'%s' connection modify '%s' +vpn.data 'username=%s'",
             esc_nmcli, esc_name, esc_user);
    free(esc_nmcli); free(esc_name); free(esc_user);
-   INF("VPN username write: %s", cmd);
+   WRN("VPN username write: %s", cmd);
 
    ctx = calloc(1, sizeof(*ctx));
    if (!ctx) { if (cb) cb(data, EINA_FALSE); return; }
@@ -206,7 +206,7 @@ enm_vpn_username_dialog(const char *conn_name, const char *type_label,
    Evas_Object *frame, *table, *label, *entry;
    char header[160];
 
-   INF("VPN username dialog for '%s' (%s)", conn_name ?: "?", type_label ?: "?");
+   WRN("VPN username dialog for '%s' (%s)", conn_name ?: "?", type_label ?: "?");
    dialog = e_dialog_new(NULL, "E", "nm_vpn_username");
    if (!dialog) { if (cb) cb(data, NULL); return; }
 
@@ -366,7 +366,7 @@ _probe_on_del(void *data, int type EINA_UNUSED, void *event)
    if (svc) { char *d = strrchr(svc, '.'); short_name = d ? d + 1 : svc; }
 
    needed = enm_vpn_username_needed(short_name, conn_type, username);
-   INF("VPN username probe result: id=%s svc=%s conn_type=%s username=%s "
+   WRN("VPN username probe result: id=%s svc=%s conn_type=%s username=%s "
        "needed=%d (blob_len=%zu)",
        id ?: "(null)", short_name ?: "(null)", conn_type ?: "(null)",
        username ?: "(null)", (int)needed, strlen(blob));
@@ -412,7 +412,7 @@ enm_vpn_username_maybe_prompt(const char *conn_name)
             "connection show '%s'",
             esc_nmcli, esc_name);
    free(esc_nmcli); free(esc_name);
-   INF("VPN username probe: %s", cmd);
+   WRN("VPN username probe: %s", cmd);
 
    ctx = calloc(1, sizeof(*ctx));
    if (!ctx) return;


-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.