<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/108685/">http://git.reviewboard.kde.org/r/108685/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 10th, 2013, 8:43 p.m. UTC, <b>Dawit Alemayehu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Why not simply use the KIO::http_post function that accepts a pointer to QIODevice ? All you have to do is set the pointer to NULL, pass the total total size to the ioslave and it should work for the scenario you outlined.
KIO::TransferJob* job = KIO::http_post(url, 0, size, flags);
job->setAsyncDataEnabled(true);
...
...
</pre>
</blockquote>
<p>On February 11th, 2013, 10:51 a.m. UTC, <b>Àlex Fiestas</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Right I coul do that, nevertheless don't you think that being able to overwrite the content-length as you do with everything else could be useful and handy?</pre>
</blockquote>
<p>On March 10th, 2013, 12:25 a.m. UTC, <b>Dawit Alemayehu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">You can override a lot of things, but not everything. Anyhow, other than adding yet another HTTP meta-data which is really not that necessary at the moment, I have no other objection to adding this particular meta-data. I simply dislike the notion of adding a meta-data just for convenience.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Well, the solution you propose is quite tricky to find :/ I will test it during this week and see how it goes.
Besides that, maybe for KF5 we should revise this API and try to make it a little bit more easy to use.</pre>
<br />
<p>- Àlex</p>
<br />
<p>On January 31st, 2013, 4:15 p.m. UTC, Àlex Fiestas wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs, Dawit Alemayehu and Andreas Hartmetz.</div>
<div>By Àlex Fiestas.</div>
<p style="color: grey;"><i>Updated Jan. 31, 2013, 4:15 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">When submitting a Content-Type: multipart/related using AsyncDataEnabled you need to specify the Content-Length of the whole thing you are trying to send, not just of the first chunk of the data. http::SendBody will append the Content-Length using the length of the first chunk of data automatically.
This patch adds the possibility of indicating the Content-Lenght using metadata "content-length".
Another option will be using the data set in "setTotalSize" in case it makes all the trip down to the kioslave.
Also, is there any case where we want AsyncDataEnabled but we don't need to specify the content-length? </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kioslave/http/http.cpp <span style="color: grey">(2d139a9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108685/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>