After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 682384 - really crappy error dialog when failing to connect anonymously to a share
really crappy error dialog when failing to connect anonymously to a share
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gvfs-maint
gvfs-maint
aklapper[errormessages]
Depends on:
Blocks:
 
 
Reported: 2012-08-21 18:52 UTC by William Jon McCann
Modified: 2012-08-24 12:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (28.38 KB, image/png)
2012-08-21 18:52 UTC, William Jon McCann
  Details
Strip off dbus stuff from error message before returning it (816 bytes, patch)
2012-08-22 20:43 UTC, William Jon McCann
none Details | Review

Description William Jon McCann 2012-08-21 18:52:48 UTC
Created attachment 222061 [details]
screenshot

When I try to connect anonymously to my NAS share I get the attached dialog. Ideally it wouldn't allow me to try it if it isn't going to work but even still we can do better with the error.
Comment 1 William Jon McCann 2012-08-21 20:28:18 UTC
<mccann> in what cases does gdbus make error strings like this? http://bugzilla-attachments.gnome.org/attachment.cgi?id=222061
<davidz> when someone forgets to call http://developer.gnome.org/gio/unstable/gio-GDBusError.html#g-dbus-error-strip-remote-error
<davidz> it's an interesting question where these calls should be inserted....
<mccann> ah! 
<mccann> I think nautilus is just getting them from gio
<davidz> I think it's probably a problem with the proxy volume monitor
<davidz> ask tomas when he's back from vacation? (I'd file a bug against gvfs now ... copy/paste this IRC conversation ... please cc me)
<davidz> (tomas recently ported the proxy volume monitor from libdbus to gdbus - bug could be introduced there)
<davidz> so... the way I think it works is that the AFP backend generates a GError with (domain=g-io-error, code=15, message = "AFP server DiskStation doesn't support anonymous login")
<davidz> which is correct
<davidz> we shove that over D-Bus and GDBus finds that g-io-error is not registered so it inserts extra information so the other end can recover the "(domain=g-io-error, code=15" part
<davidz> we just need to strip that extra information... it's that simple I think
* davidz must have overlooked this when reviewing tomas' patches
<davidz> mccann: anyway, I would file a bug against gvfs for this
Comment 2 William Jon McCann 2012-08-22 20:43:13 UTC
Created attachment 222192 [details] [review]
Strip off dbus stuff from error message before returning it