You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bugs@apr.apache.org by bu...@apache.org on 2008/04/06 20:35:47 UTC

DO NOT REPLY [Bug 44761] New: apr_uri_parse doesn't set path to "/" for ' http://hostname' url.

https://issues.apache.org/bugzilla/show_bug.cgi?id=44761

           Summary: apr_uri_parse doesn't set path to "/" for
                    'http://hostname' url.
           Product: APR
           Version: HEAD
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P3
         Component: APR-util
        AssignedTo: bugs@apr.apache.org
        ReportedBy: lgo@mobsol.be


A user of Subversion reported an issue where svn doesn't work on repositories
hosted directly under the root of the server. This issue is tracked as #3167
(http://subversion.tigris.org/issues/show_bug.cgi?id=3167).

The cause of the issue seems to be apr-util's apr_uri_parse function that
doesn't behave as documented:

Snippet of code that shows the issue:
----
repos_url = "http://localhost";
apr_status_t status = apr_uri_parse(pool, repos_url, &uri);
assert(uri.path[0] == '/' && uri.path[1] == '\0');
----

Here uri.path should be '/', as is stated in the documentation of the apr_uri_t
structure in apr_uri.h:

struct apr_uri_t {
  ..
  /** the request path (or "/" if only scheme://host was given) */
  char *path;
  ..
}

I have a patch ready about which I have some questions, so I'll send it to
apr-dev (and include the archive url here in the issue).


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org


DO NOT REPLY [Bug 44761] apr_uri_parse doesn't set path to "/" for 'http://hostname' url.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=44761


Eric Covener <co...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #4 from Eric Covener <co...@gmail.com>  2009-01-07 12:44:51 PST ---
Applied Dan's patch documenting/testing current behavior back through 1.3.x in
r732476 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732476 )r732477 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732477 )r732478 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=732478 )

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org


DO NOT REPLY [Bug 44761] apr_uri_parse doesn't set path to "/" for 'http://hostname' url.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=44761


Dan Poirier <po...@pobox.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |poirier@pobox.com
             Status|NEEDINFO                    |NEW
           Keywords|                            |PatchAvailable




-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org


DO NOT REPLY [Bug 44761] apr_uri_parse doesn't set path to "/" for 'http://hostname' url.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=44761





--- Comment #3 from Dan Poirier <po...@pobox.com>  2008-10-16 11:21:37 PST ---
Created an attachment (id=22745)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=22745)
Patch to fix the comment and add a test for the current behavior.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org


DO NOT REPLY [Bug 44761] apr_uri_parse doesn't set path to "/" for 'http:// hostname' url.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=44761


Will Rowe <wr...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO




--- Comment #2 from Will Rowe <wr...@apache.org>  2008-05-02 11:02:36 PST ---
As I recall the list dialog, the behavior was by design, the URI does not
contain a path so should not have a default path.

But as you observe, the docs are wrong.  Care to submit a revised patch for
the mis-documented member?


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org


DO NOT REPLY [Bug 44761] apr_uri_parse doesn't set path to "/" for 'http:// hostname' url.

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=44761


Lieven Govaerts <lg...@mobsol.be> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://www.mail-
                   |                            |archive.com/dev%40apr.apache
                   |                            |.org/msg19732.html




--- Comment #1 from Lieven Govaerts <lg...@mobsol.be>  2008-04-06 12:53:01 PST ---
Patch sent to apr-dev, see url for thread in archive.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org