This has been reported in:
http://drupal.org/node/61528#comment-158819
http://drupal.org/node/30525#comment-166465

This all deals with the same issue: IE is picky about WHAT headers it gets for files and the ORDER of those headers.

In this case, 2 windows appear with the document in both.

This has been tested on:
IE/XP with Office 2003
FF/XP with Office 2003
FF/Linux

hosted on Apache2/PHP4.3/Drupal 5.02beta on Centos 4.4

Enjoy!

CommentFileSizeAuthor
#1 upload_21.patch887 bytesRobRoy
upload.module.patch_3.txt779 bytesdavea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RobRoy’s picture

FileSize
887 bytes

Bah, my last message got deleted. Anyways, the patch needs some work. It adds back stuff that we removed in one issue above and some other stuff that's probably unneeded. I stripped down the headers to the bare bones and it looks like this makes PDFs behave EXACTLY the same in public/private files (open up in the current window).

I used web-sniffer.net to see what headers are sent for a public PDF and it looked like just Content-Length and Content-Type were the relevant headers and no name was used for Content-Type (which was causing errors with IE6 when the filename had spaces in it). None of the other stuff. So let's work up from this point to see if any other headers are truly needed for private files.

Please test this out on a variety of private files and maybe this is enough.

RobRoy’s picture

Version: 5.0-beta2 » 5.x-dev

Correcting version. Also, Dave there were some code-style/spacing issues in your patch FYI. Thanks for spearheading this issue, let me know how the latest patch works for you.

drumm’s picture

Status: Needs review » Needs work

The last follow-up with a patch mentioned it needs work.

RobRoy’s picture

Status: Needs work » Needs review

Actually, I was saying Dave's patch had some issues. The patch I posted in #1 is up for review.

Dries’s picture

Status: Needs review » Fixed

Yes, this patch is needed to make things work. Committed.

Anonymous’s picture

Status: Fixed » Closed (fixed)