Comment 1 for bug 401554

Revision history for this message
Eledran (eledran-deactivatedaccount) wrote :

I think that rendering the page to x first lines of the diff is not the wiser solution. Neither to render it to x megabytes of text.

Instead, give a link to the diff could be given, with the mask (Download diff {Size in MB}), as even the diff lines counter for the raw text file can run out of time. Besides the script can be very optimized, a attacker could down the site by making it run many times...

If you want, you can try to give a look around this set of diffs:

http://bazaar.launchpad.net/~icy-phoenix-admins/icy-phoenix/1.2-stable/revision/1?start_revid=110

Warning! Even my browser (firefox) has said me the script used for the diff retrive was blocking the computer.

So, there can be a server-side script that makes the diff file, and the lp template files could strip out the diff from the page, showing only the link, and its size in megabytes.

Just tring to give ideas as the fix is not commited yet... but as you are holding the fix development, I guess it is up to you and the Launchpad groups up to determine the appropiate fix for this bug.