65024 - "Save As" doesn't do basic auth, doesn't give good error.

Status: VERIFIED DUPLICATE of bug 116279

(Core Graveyard :: File Handling, defect, P3)

Description

[Anthony Baxter]

For an example: go to
http://www.acm.org/pubs/citations/journals/cacm/1999-42-10/p72-cusumano/

Shift-Click on 'PDF' - I get the 'Save As' file chooser,
then a popup with

Unknown error [1 80004005]

Just clicking normally gives the basic auth popup. As far as
I can see, 'Save As' doesn't have the basic auth stuff hooked up.

This is with built 2000-12-26, on Linux

Comment 1

[Darin Fisher]

this is not an http bug.  the implementor of "save as" is failing to specify
an nsIPrompt on the http channel, thereby leaving http with no way to do basic
auth.

->mscott (does this belong to the uriloader?)

Comment 2

[Keyser Sose]

WORKSFORME
Platform: PC
OS: Linux 2.2.16
Mozilla Build: 2001010908


Reporter try one of the latest nightlies and report back if its a problem or
not. My bet is that one of the latest nightles and/or a new profile will fix the
problem.

Comment 3

[Jesse Ruderman]

I see this bug on Win98 with 2001 011204, confirming and setting OS to All.

Comment 4

[Anthony Baxter]

I just tried the same version as you (linux, 2001010908) on Redhat 6.2,
and I see the same, broken, behaviour. Shift-Click on the 'PDF' icon
(or select 'Save Link As' on it), get file selector, pick file, get
"unknown error" popup.

I also tried with a new profile, still see it.

Comment 5

[hirata masakazu]

*** Bug 67568 has been marked as a duplicate of this bug. ***

Comment 6

[hirata masakazu]

"Save As" is not intended to mean as "Save Link As", by the way.
Save As is to save the page source (yes, there is another bug for that), and
Save Link As is to save the pdf file in this case. Save Link As is supposed to
bring up basic authentication which is skipped here.

Comment 7

[Scott MacGregor]

this is actually Bill's save as code from the file menu i believe.

Comment 8

[Paul Chen]

Marking nsbeta1+, p3, mozilla0.9

Comment 9

[Bill Law]

I'm not getting to this dialog till next round...

Comment 10

[Viswanath Ramachandran]

as discussed in team meeting, moving all Nav+ team members nsbeta1+ P3 bugs from 
mozilla0.9.1 to mozilla0.9.2. 

Comment 11

[Paul Chen]

nav triage team:

Pushing out to mozilla0.9.3

Comment 12

[Paul Chen]

nav triage team:

Nice to have, but not a mozilla0.9.3 stopper. Marking mozilla1.0

Comment 13

[benc]

-> xp apps, out of networking 
workaround might be to auth before you hit the link.

Comment 14

[Anthony Baxter]

That workaround isn't always going to be useful. The example 
given is a site where you can see the abstracts, but it's only
when you try to download the paper that the auth is required.
Mozilla seems to be about the only browser that I can find that
doesn't get this right ... :(

Comment 15

[sairuh (rarely reading bugmail)]

spam: over to File Handling. i have not changed the assigned developer [or the
other fields for that matter], so if anyone realizes that a bug should have a
more appropriate owner, go ahead and change it. :)

Comment 16

[Paul Chen]

->law

Comment 17

[Bill Law]

->mozilla0.9.9

Comment 18

[Bill Law]

I need an update on this bug.  Currently, that URL doesn't have a link to a PDF 
file.  There's a text link that says "Click here to access the full text" and 
that opens another web page with a login screen.  We handle clicking on that 
link the same as IE.  I don't have an account but I suspect logging in would 
work OK.

I don't think we've done anything to fix the problem Darin alluded to (not 
implementing nsIPrompt) so there still may be a problem here.  But I think 
maybe we get some default nsIPromptService if there isn't one specific to the 
http request, so maybe there is no longer any bug here.

Please check it out and let me know.

Comment 19

[Boris Zbarsky [:bzbarsky]]

The acm site no longer uses basicauth.  Changing url to a working testcase.

This is still broken.

Comment 20

[Boris Zbarsky [:bzbarsky]]

putting username/password to use in status whiteboard.  Sorry about forgetting
that....

Comment 21

[Bill Law]

This needs to be done in nsWebBrowserPersist.

Comment 22

[Adam Lock]

*** This bug has been marked as a duplicate of 116279 ***

Comment 23

[sairuh (rarely reading bugmail)]

v