[Edbrowse-dev] Rebundle

[Karl Dahlke <eklhad@comcast.net>]

> Here's what I have for the native implementation of a javascript function.

Kevin I finally got round to looking at this.

You should free the strings created by js_c_str() (memory leak).
You may also have to free serverData after it is converted to a js string.
It too is allocated.
javatext should be initialized to null else
the last line will seg fault when httpConnect fails.
Not clear what happens for other error codes besides 200.

That said, it's a great and simple implementation of
a synchronous http call.
And with my latest commits,
it will use the proxies and agent and cookie jar in the config file.
But it will not use transient or new cookies, like session cookies,
that might be produced by the initial web page fetch from edbrowse,
and it won't until we merge things together and there is but one instance of curl running.

Next question is could we spin off the http in the background,
which edbrowse indeed can do,
and how that would fit together,
and I'll think about that one,
but meantime I do think some is better than none,
and it might be worth implementing this synchronous routine,
in 30 lines of code, so more websites work,
and then work on the asynchronous model later.
But that's just my opinion / philosophy,
which is not always right.
A website indeed could work, that didn't work before,
but you may have to wait a few seconds for the http to return serially -
well I'd make that tradeoff for now.

anyways if you want to send me a revised function, or patch, or whatever...

Karl Dahlke