Hacker News Reader

Comment by irrationaljared | Hacker News Reader

Fair enough. Feature added.

There's now a link next to the "Download Photo" button called "View Original" when you click to view the larger photos.

Update

Unfortunately on Chrome this link doesn't work because the photo is stored in a way that tells Chrome to automatically download the photo. I'll have to dig in to figure out if there is a way to force the photo to be displayed instead of automatically downloaded.

View on HN

Are your originals actually the original files? Some people have strong opinions about data leakage via EXIF metadata, specifically location and sometimes date/time. Do you strip that information out during upload/resizing?

Overall, it looks great. My family would definitely use it, for similar reasons to yours. Nice work!

The original photos are processed for more efficient storage, so I believe that all EXIF data is stripped out, but I'll have to check to make sure.

The code/library/service responsible for storing the original file on S3 is most likely setting "response-content-disposition" to "attachment" when it stores the file on S3. That tells S3 to deliver the file with "Content-disposition: attachment" in the HTTP headers which is what causes browsers to save to disk instead of display.

You want "Content-disposition: inline" or no content-disposition header.

I was able to fix this. You should be able to view the photos via the link now, but if anyone has trouble with it please let me know.

Reply on news.ycombinator.com