[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