85740 - Can Not Post Binary Data

Status: VERIFIED INVALID

(Core Graveyard :: Plug-ins, defect, P2)

Description

[lmcquarr]

A new feature of Acrobat 5 requires the ability to post binary data.
This works in Nav 4.X and IE, but it does not work in Mac IE 5.X or in
mozilla.  It appears that mozilla and Mac IE 5 open the post data
as ASCII, and the data, therefore, is truncated at the first null.

To repo:

1 - Install Acrobat 5.
2 - Install mozilla and drop the nppdf32.dll plug-in in the mozilla plug-ins
directory.
3 - Goto the above URL and press the submit button (you can digitally sign
it first, if you wish).

Expected result:

PDF file is echod back to the browser and is displayed in Acrobat.
Compare with behavior of Nav 4.X.)

Actual result:

Acrobat error:  File must begin with "%PDF"


(

Comment 1

[Peter Lubczynski]

cc:ing Eric Pollmann for input.

Eric, how do forms handle POSTing of binary data files? Is there a difference? 

Most of the plugin posting code is located near:
http://lxr.mozilla.org/seamonkey/source/modules/plugin/nglsrc/nsPluginHostImpl.cpp#4433

Comment 2

[Peter Lubczynski]

Liz, how bad is this bug? 

Slotting into mozilla0.9.3 as P2 due to PDT+.

Note: Eric is on vacation till July 9th. 

Comment 3

[lmcquarr]

We can live with this bug :-( for now.  (Mac IE 5 doesn't work either.)

Comment 4

[av (gone)]

BTW, I see the same error with Navigator 4 on NT.

Comment 5

[av (gone)]

Attachment: some code which I think should be here anyway

Comment 6

[av (gone)]

Liz, it happened that posting data was not implemented in full-page case 
(nsPluginViewer.cpp). In the patch above I added what I think should do the job. 
With this patch there is no more error message but from what I see I cannot 
determine whether it goes right or not. So, if you could apply this patch and 
try it...

Comment 7

[Peter Lubczynski]

Liz can't verify because all docuemnts off of access.adobe.com see to come up
blank. 

However, that really does look like it should be there. r=peterl

Comment 8

[Marek Z. Jeziorek]

Andrei, this is PDT+ bug. Tomorrow, on Tuesday, we'll try to build the first RTM
candidate. It would be good, if this could be resolved ASAP.

Comment 9

[lmcquarr]

Andrei --

I can test your patch. (It looks like if I disable the proxy server
then my documents do not come up blank in mozilla, at least on a clean
lab machine ... still investigating).  

Comment 10

[lmcquarr]

Sorry, but the patch did not work.  :-(


Also, Netscape 4.X on NT *does* work (the only other browser
that currently does not work is Mac IE 5).

Comment 11

[Marc Attinasi]

I'm thinking this is not really PDT+ - Liz thinks we can live with it for now,
but lchiang marked it PDT+ on the 28th. Can we re-evaluate the urgency?

Comment 12

[lchiang]

I probably marked this PDT+ due to some emails going to PDT+.  If this is not a
stopper, per se, for Liz and company, then pls feel free to remove the PDT+. Thanks.

Comment 13

[av (gone)]

Removing PDT+ as per attinasi suggestion.

Comment 14

[lmcquarr]

In the process of compiling a list of known issues with NS 6.1 and Acrobat,
I have been reviewing open bugs that I have submitted
and I have determined that this bug is invalid.

The actual cause of the problem with SubmitAsPDF.pdf is not
that it can not post binary data, but that mozilla is doing
a GET rather than a POST in this case when we call
NPN_PostURL.  I have already entered that as a bug: 77603. 

This bug, therefore, can be withdrawn.

Comment 15

[shrirang khanzode]

v