You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by "Tommy Knowlton (JIRA)" <ji...@apache.org> on 2008/05/07 19:08:56 UTC

[jira] Closed: (BUILDR-67) download(art => 'http://example.com/foo?query') fails to send query part of URI when performing the HTTP GET

     [ https://issues.apache.org/jira/browse/BUILDR-67?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tommy Knowlton closed BUILDR-67.
--------------------------------


I'm assuming that as the 'reporter' I am also expected to close the issue once I see it resolved.
If that is incorrect workflow, I hope to be told what the correct workflow would be.

I've enjoyed learning how to contribute to this project, thanks for providing the git mirror.

> download(art => 'http://example.com/foo?query') fails to send query part of URI when performing the HTTP GET
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: BUILDR-67
>                 URL: https://issues.apache.org/jira/browse/BUILDR-67
>             Project: Buildr
>          Issue Type: Bug
>          Components: Core features
>    Affects Versions: 1.2.10
>            Reporter: Tommy Knowlton
>             Fix For: 1.3.1
>
>         Attachments: 0001-Add-a-failing-spec-test-demonstrating-issue-BUILDR.patch, 0002-fix-for-BUILDR-67.patch
>
>   Original Estimate: 0h
>  Remaining Estimate: 0h
>
> This appears to be caused by inappropriate use of the URI's path member at the point where the GET is performed.
> I have already tested a fix that works against 1.2.10 (released) code.
> Also, looking at the trunk code, it appears that this bug remains, although the code affected in 1.2.10 has since been rewritten to use Net::HTTP (i.e., the rewrite still uses uri.path when uri.request_uri seems to be more correct).
> I *have not* yet created a unit test to fail.
> I'm not clear in my mind about the 'priority' on this, but I'm leaving it at 'Major' for now, because it breaks my organization's builds. I don't particularly care if the project management team sees fit to move the priority down on this.
> I expect to submit one or more patches Real Soon Now (TM).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.