<table><tr><td style="">fvogt added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D23384">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Clicking a http link in my chat app now kiofuses it and then has the browser desperately try to open /run/user/1000/kio-fuse-bla/http/kde.org/ - it only gets the first character in the file, so I usually just see < for a HTML page</p></blockquote>
<p>Ignoring the elephant in the room which is that this diff forces everything through <tt style="background: #ebebeb; font-size: 13px;">mountUrl</tt>, that's the expected behavior with a plain HTTP URL as the size isn't known until the file is downloaded. So <tt style="background: #ebebeb; font-size: 13px;">stat</tt> reports a size of <tt style="background: #ebebeb; font-size: 13px;">1</tt> until the file is actually opened.<br />
This is unavoidable, otherwise every <tt style="background: #ebebeb; font-size: 13px;">ls</tt> would trigger a download of all files. If handling this better is important, <tt style="background: #ebebeb; font-size: 13px;">HTTPProtocol::stat</tt> could use a <tt style="background: #ebebeb; font-size: 13px;">HEAD</tt> request to get the <tt style="background: #ebebeb; font-size: 13px;">Content-Length</tt>, but that doesn't work in all cases either.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23384">https://phabricator.kde.org/D23384</a></div></div><br /><div><strong>To: </strong>feverfew, fvogt, davidedmundson, dfaure, ngraham<br /><strong>Cc: </strong>broulik, sitter, davidedmundson, kde-frameworks-devel, ngraham, LeGast00n, GB_2, michaelh, bruns<br /></div>