Re: [Libguestfs] [libnbd PATCH 0/2] Tighten URI parser

On Tue, Jun 25, 2019 at 09:09:58PM -0500, Eric Blake wrote: Oh, I get it now. So "nbd:unix:/path" is a valid URL without authority where the path is "unix:/path". Well, TIL =) It is nicely visible here: https://tools.ietf.org/html/rfc3986#section-3 These patches are fine from my standpoint, although I must say thinking about the above as defined in the RFC makes me re-evaluate my feelings about it. Having URI without authority just use a local file (socket) makes sense now. Even having the unix: prefix to show that it is a UNIX socket seems sensible. At least unless you realize that having an actual URI it makes sense to have that as a part of the scheme, as is customary. What I am trying to say is that maybe just the first patch is enough. When people want to just write, for example nbd:/tmp/my_nbd.sock by hand, it is easier than typing the whole nbd+unix:?socket=/tmp/my_nbd.sock. On the other hand, how often do you need to write it manually... Anyway, I don't really have strong feelings about this, so whatever you choose is fine as long as nbd:unix:/path does not try connecting to localhost over network =) The second one should say "Reject nbd:/path/to/socket as invalid URL", I guess. Thank you.