From: Tom Lane <tgl@sss.pgh.pa.us>
To: Magnus Hagander <magnus@hagander.net>
cc: Greg Stark <stark@mit.edu>, Peter Geoghegan <peter@2ndquadrant.com>,
	PostgreSQL WWW <pgsql-www@postgresql.org>
Subject: Re: New archives for testing
In-reply-to: 
 <CABUevEznc2gdDdOEqDkNM4sjXWwQMvDWvE6j0_8TAyf3+=ZEZQ@mail.gmail.com>
References: 
 <CABUevEwmR5+9fc1i1cFVpc1S8XShHwvKopcaKKfq-Txa+_mq_g@mail.gmail.com>
	<18383.1357499527@sss.pgh.pa.us>
	<CABUevEzx8tAUZErb8PhADj=mzSEUdUE8-ueqUfuP=mnNjbMQ3A@mail.gmail.com>
	<CAEYLb_UWpcNdyEHzOyzGYs9+CEXqrNP8-c5tPd6X9v1fK9jKYQ@mail.gmail.com>
	<CABUevExrSBxtW3qkUA7-cn=DFTVVdcahDpWFovMcWOnRxf=ZhQ@mail.gmail.com>
	<CAM-w4HOUE2v_d1xhZOTk930ugu2UiAQbHLy41DjUiZtT355KAg@mail.gmail.com>
	<CABUevEznc2gdDdOEqDkNM4sjXWwQMvDWvE6j0_8TAyf3+=ZEZQ@mail.gmail.com>
Comments: In-reply-to Magnus Hagander <magnus@hagander.net>
	message dated "Sun, 06 Jan 2013 22:12:44 +0100"
Date: Sun, 06 Jan 2013 16:25:08 -0500
Message-ID: <6904.1357507508@sss.pgh.pa.us>
Precedence: bulk
Sender: pgsql-www-owner@postgresql.org

Magnus Hagander <magnus@hagander.net> writes:
> On Sun, Jan 6, 2013 at 9:59 PM, Greg Stark <stark@mit.edu> wrote:
>> I would be more concerned with the file size. You don't want someone
>> to be able to make an archive page send a hapless viewer a 100M image
>> file. Luckily file size doesn't require parsing the meta data.
>> 
>> Can you not deal with the image size by just sticking a css attribute
>> for max-height and max-width on the inlined image?

> That's certainly another option - limit it by size that way, and then
> limiting it by filesize for whether we include it or not. We can even
> give it scrollbars, I guess. That does make even more sense.

A file-size-based limit on what we'll inline seems perfectly sensible
from here.  As for scrollbars, if it's big enough to need those, we
probably don't want it inline ...

			regards, tom lane


-- 
Sent via pgsql-www mailing list (pgsql-www@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-www