summaryrefslogtreecommitdiff
path: root/src/bus-proxyd/proxy.c
diff options
context:
space:
mode:
authorDavid Herrmann <dh.herrmann@gmail.com>2015-02-14 19:01:21 +0100
committerDavid Herrmann <dh.herrmann@gmail.com>2015-02-14 19:01:21 +0100
commite28569311f5385cde76e4b84adbec6609b451cf9 (patch)
treec1f418cfd6b3b8b00cf01a52014238abfb8dadbd /src/bus-proxyd/proxy.c
parent749806b389f53f7dd43a2cae1589e63dddf51990 (diff)
bus-proxy: be more verbose if messages cannot be forwarded
If we cannot forward messages, include information on the peer and message just like the xml-policy does. This helps debugging such situations and figuring out what exactly is going wrong.
Diffstat (limited to 'src/bus-proxyd/proxy.c')
-rw-r--r--src/bus-proxyd/proxy.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/src/bus-proxyd/proxy.c b/src/bus-proxyd/proxy.c
index 4be2a6289a..3dea908f5b 100644
--- a/src/bus-proxyd/proxy.c
+++ b/src/bus-proxyd/proxy.c
@@ -729,7 +729,10 @@ static int proxy_process_destination_to_local(Proxy *p) {
/* Return the error to the client, if we can */
synthetic_reply_method_errnof(m, r, "Failed to forward message we got from destination: %m");
- log_error_errno(r, "Failed to send message to client, ignoring: %m");
+ log_error_errno(r,
+ "Failed to forward message we got from destination: uid=" UID_FMT " gid=" GID_FMT" message=%s destination=%s path=%s interface=%s member=%s: %m",
+ p->local_creds.uid, p->local_creds.gid, bus_message_type_to_string(m->header->type),
+ strna(m->destination), strna(m->path), strna(m->interface), strna(m->member));
return 1;
}
@@ -799,7 +802,10 @@ static int proxy_process_local_to_destination(Proxy *p) {
return 1;
synthetic_reply_method_errnof(m, r, "Failed to forward message we got from local: %m");
- log_error_errno(r, "Failed to send message to bus: %m");
+ log_error_errno(r,
+ "Failed to forward message we got from local: uid=" UID_FMT " gid=" GID_FMT" message=%s destination=%s path=%s interface=%s member=%s: %m",
+ p->local_creds.uid, p->local_creds.gid, bus_message_type_to_string(m->header->type),
+ strna(m->destination), strna(m->path), strna(m->interface), strna(m->member));
return 1;
}