Re: a bug for curl-loader & some more question, thanks for all you

[Robert I. <cor...@gm...>]

Hi Wei Guo,

Thank you for the patches.

I'll review them later.

Please, post and send everything to cur...@li...
Thanks,
Robert

2011/9/23 wei guo <wei...@gm...>

> hi, Robert:
> The attachment is the patches for 0.53,
>
> 1. fix the statement which check cycle time between use_current_url and its
> primary url.
> 2. add HEAD & DELETE method support.
>
> I test the feature above, It works well.
>
> shall we add openssl-dev in packages of curl-loader?  adding it can make
> user more convenient.
>
> additionally, I used curl-load 0.53, after I have compile it, It could not
> work well, the error log is:
>      bind failed with errno 22: Invalid argument
> I trace the error log, It's in lib/curl/ dir. I use old version
> curl-7.19.7.tar.bz2 to replace new version. It works well.
> Maybe It's the bug of new version curl, I'm not sure the reason. when I
> find the reason, I will mail you.
>
> I'm very glad to receive your reply letter, Thank you.
>
> Best Regards.
>
>
> 在 2011年9月22日 下午2:13，Robert Iakobashvili <cor...@gm...>写道：
>
> Dear Guo Wei,
>>
>> Thanks you for comments and issues described.
>>
>> Please, submit your proposed corrections/extensions
>> to this list cur...@li... as patches for
>> 0.53, a separate patch per issue (diff -Naru).
>>
>> You can join the list here:
>> https://lists.sourceforge.net/lists/listinfo/curl-loader-devel
>>
>> Openssl is normally exists at a linux development machine,
>> and what is necessary is to add openssl-dev (on debian).
>>
>> Regards,
>> Robert Iakobashvili, Ph.D.
>>
>> Home:       http://www.ghotit.com
>> Blog:         http://dyslexia-blog.ghotit.com
>> Twitter:      http://twitter.com/ghotit
>> Facebook: http://facebook.com/ghotit
>> ......................................................................
>>                   Ghotit Dyslexia
>> Assistive technology that understands you
>> ......................................................................
>>
>>
>> 2011/9/22 wei guo <wei...@gm...>
>> >
>> > hi，curl-loader developers:
>> >
>> > First, I wish to express my appreciation for your work on curl-loader,
>> It help me to solve big problem on my project. I have a bug and some more
>> question to show you.
>> >
>> > Bug:
>> > 1. In curl-loader-0.52 version, in file parse_conf.c line 2404,  the
>> statement :
>> >               if (url_m->url_dont_cycle != url_m->url_dont_cycle)
>> > It used to check cycling or not-cycling status of the CURRENT_URLs is
>> the same as for the primary-URL,  obviously, program can never run into this
>> section, This problem still remain in curl-loader-0.53, you can simply
>> change the first url_m or the second to url, that can modify this problem.
>> i.e
>> > if (url_m->url_dont_cycle != url->url_dont_cycle)
>> >
>> > Additional, I have some question about curl-loader：
>> > 1. Why we don't add openssl package to our curl-loader package？If we add
>> it, wo don't need to change openssldir.sh or set OPENSSLDIR environment
>> variable，after modify Makefile，just one command. $make, you can get
>> curl-loader. :)
>> >
>> > 2. Why our main funciton need to run as the root  user? I know is used
>> to add secondary ip to local machine, but I think add secondary ip , if we
>> needed,  is the work before the load test run. Curl-loader run as normal
>> user is more safety. If you  need to change the environment vriable, just do
>> it before the load test begin. Here I think we can delete the check of root
>> permission. The first time I user curl-loader, I run as root, and not very
>> clear about curl-loader will add secondary ip on my local machine, thus my
>> workmate could not visit internet.
>> > 3. Curl-loader  now olny support  PUT, GET, POST method, I add HEAD &
>> DELETE method for it. gratefully, the program structure is very clear,  It
>> is an easy task to finish it.  Your work benefit me a lot, if possible, I
>> hope I can contribute my code to this project  and give others help. How
>> could I joined this project?
>> >
>> > maybe I don't describe very clear, if some place is not distinct, please
>> mail me: wei...@gm...
>> >
>> > At last, Thank you all for you work once more.
>> >
>> > Regards,
>> >
>> > Guo_wei
>>
>
>


-- 
Regards,
Robert Iakobashvili, Ph.D.

Home:       http://www.ghotit.com
Blog:         http://dyslexia-blog.ghotit.com
Twitter:      http://twitter.com/ghotit
Facebook: http://facebook.com/ghotit
......................................................................
                   Ghotit Dyslexia
Assistive technology that understands you
......................................................................