Confirmed. Thank you!

Alexander.

On Wed, Feb 10, 2010 at 01:07, Fabio Mascarenhas <mascaren...@acm.org> wrote:
> Done!
> --
> Fabio Mascarenhas
>
> On Tue, Feb 9, 2010 at 6:51 PM, Alexander Gladysh <aglad...@gmail.com>
> wrote:
>>
>> > Also, I think I've found a bug in your patch:
>>
>> > index c778006..8386197 100644
>> > --- a/etc/hackedwsapi/wsapi/xavante.lua
>> > +++ b/etc/hackedwsapi/wsapi/xavante.lua
>> > @@ -38,7 +38,7 @@ local function set_cgivars (req, diskpath,
>> > path_info_pat, script_name_pat, extra
>> >       CONTENT_LENGTH = req.headers ["content-length"],
>> >    }
>> >    if req.cgivars.PATH_INFO == "" then req.cgivars.PATH_INFO = "/" end
>> > -   for n,v in ipairs(extra_vars or {}) do
>> > +   for n,v in pairs(extra_vars or {}) do
>> >      req.cgivars[n] = v
>> >    end
>> >    for n,v in pairs (req.headers) do
>>
>> Fabio, can you please commit this fix (if it is valid, of course)?
>>
>> I'd like to continue to use wsapi-xavante from cvs rockspec, now it
>> lives outside of luarocks ecosystem due to this bug...
>>
>> I may send you a GitHub pull request if it would be more comfortable.
>>
>> Thank you,
>> Alexander.
>>
>> _______________________________________________
>> Kepler-Project mailing list
>> Kepler-Project@lists.luaforge.net
>> http://lists.luaforge.net/cgi-bin/mailman/listinfo/kepler-project
>> http://www.keplerproject.org/
>
>
> _______________________________________________
> Kepler-Project mailing list
> Kepler-Project@lists.luaforge.net
> http://lists.luaforge.net/cgi-bin/mailman/listinfo/kepler-project
> http://www.keplerproject.org/
>

_______________________________________________
Kepler-Project mailing list
Kepler-Project@lists.luaforge.net
http://lists.luaforge.net/cgi-bin/mailman/listinfo/kepler-project
http://www.keplerproject.org/

Reply via email to