Re: [naviserver-devel] driver changes, please check

2010-04-13 Thread Gustaf Neumann
Am 12.04.10 15:10, schrieb Stephen Deasey:
> You are going to have to merge the two heads with 'hg merge' and then
> push that changeset. See: hg help merge.
>
many thanks! should be fine now.
-gustaf


--
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
naviserver-devel mailing list
naviserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/naviserver-devel


Re: [naviserver-devel] driver changes, please check

2010-04-13 Thread Gustaf Neumann
Am 12.04.10 15:18, schrieb Stephen Deasey:
> How about implementing chunked uploading?  It is a requirement of the
> HTTP 1.1 spec.
>
i have implemented a version of this and committed and pushed it to the tip.

Provide Chunked Encoding support for incoming requests
   * Shortcomings:
  a) for chunked encodings, spooling to files is turned off (since 
decoding happens in memory)
  b) chunked encodings still require X-Expected-Entity-Length to 
terminate reliably

Checking for eof would be the best for (b).

-gustaf neumann

--
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
naviserver-devel mailing list
naviserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/naviserver-devel


Re: [naviserver-devel] driver changes, please check

2010-04-12 Thread Stephen Deasey
On Sun, Apr 11, 2010 at 4:42 PM, Gustaf Neumann  wrote:
>
> The current solution works, but returns the chunked markup
> (which is fine for me, doing the chunked-decode in Tcl),
> but should done in (similar to the chunked encoding for forms).


This is not going to work reliably because of the following, right?

http://bitbucket.org/naviserver/naviserver/changeset/a1ccb0371eee/#chg-nsd/driver.c_newline2239

if (reqPtr->avail > reqPtr->expectedLength) {
/*
 * Chunk encoded data is always larger than the expected
 * length; we hope that we have sufficient data collected
 */
reqPtr->length = reqPtr->avail;
} else {



> Actually, there must be a better way for this than the following
> change: http://bitbucket.org/naviserver/naviserver/changeset/a1ccb0371eee/


How about implementing chunked uploading?  It is a requirement of the
HTTP 1.1 spec.

There is already full support for writing chunked, with tests, and
there is support for parsing mime parts in the form parsing code.

--
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
naviserver-devel mailing list
naviserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/naviserver-devel


Re: [naviserver-devel] driver changes, please check

2010-04-12 Thread Stephen Deasey
On Sun, Apr 11, 2010 at 4:42 PM, Gustaf Neumann  wrote:
>
> PS: i had to do a "hg push -f ...". Not sure, if this is intentional.
> Do the commits to tip look right?


All that's happened here is that you've made changes to your checkout
without first updating it. In the meantime, Zoran has made changes and
pushed them.

If this were CVS it would refuse to let you commit. With mercurial
obviously you can commit as it's local, but it will not let you push
and will give you a warning. You ignored the warning by using -f  :-)

You are going to have to merge the two heads with 'hg merge' and then
push that changeset. See: hg help merge.

--
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
naviserver-devel mailing list
naviserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/naviserver-devel


[naviserver-devel] driver changes, please check

2010-04-11 Thread Gustaf Neumann
Dear driver experts,

i experienced problems with PUT operations with chunked
encodings. The existing code seems to rely on provided
content-lengths for client requests with content, which is
not available form e.g. PUT operations with chunked
encoding (e.g. issued from Finder under Mac OS X).
I would like to read such requests (which come with
"connection: close") until eof, but found no easy way for
do so.

With the original code, the content length for such requests
was always 0.

The current solution works, but returns the chunked markup
(which is fine for me, doing the chunked-decode in Tcl),
but should done in (similar to the chunked encoding for forms).

Actually, there must be a better way for this than the following
change: http://bitbucket.org/naviserver/naviserver/changeset/a1ccb0371eee/

all the best
-gustaf neumann

PS: i had to do a "hg push -f ...". Not sure, if this is intentional.
Do the commits to tip look right?


--
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
___
naviserver-devel mailing list
naviserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/naviserver-devel