[Bug 743195] uri: Add parsing unit test based on GNet
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Tue Jan 20 05:40:55 PST 2015
https://bugzilla.gnome.org/show_bug.cgi?id=743195
GStreamer | gstreamer (core) | unspecified
--- Comment #5 from David Waring <david.waring at rd.bbc.co.uk> 2015-01-20 13:40:50 UTC ---
I'm having a look at this.
Yes, the parsing code assumes that a path follows an authority part, that's
easily modified to terminate the authority part at a query or fragment too, so
scheme://user@host:123?query and scheme://user@host:123#fragment would also
work.
The default was to leave the square brackets in the host string, I can change
that to strip them. Although it's debatable how to handle
"scheme://[01:23:45:67:89:ab:cd:ef:123/path" as the RFC says there must be a
closing square bracket, and that the brackets only contain unreserved,
sub-delim and ":". This means that the authority part would end at the "/". I'd
say in that case it would be host="[01:23:45:67:89:ab:cd:ef", port=123 (since
we're back to <host>:<port>) and path="/path" as the only sensible way to
interpret this.
I'll work my way through the test cases and post a patch to fix any problems I
find.
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list