Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

"Save Link As" doesn't follow redirects #11788

Closed
jonathansampson opened this issue Nov 4, 2017 · 2 comments
Closed

"Save Link As" doesn't follow redirects #11788

jonathansampson opened this issue Nov 4, 2017 · 2 comments

Comments

@jonathansampson
Copy link
Collaborator

Description

The Save Link As option doesn't follow redirects, and thus will attempt to download the intermediary file, rather than the end of a redirect chain.

For instance, assume the following: <a href="download.php?file=18112">Download PDF</a>. If the download.php file uses a location header to redirect to a PDF, Brave will download the HTML response for download.php, rather than downloading the PDF in the location header.

Steps to Reproduce

TBD: Need to find a public example. Only repro site I have at this time requires authentication.

Actual result:
HTML of response is downloaded.

Expected result:
If a location header is present, it will be followed.

Reproduces how often:
100%

Brave Version

0.19.80

about:brave info:

Brave: 0.19.80 
rev: 7d07299a5d462d3c9ae32cfbfbe7296cc57c89b9 
Muon: 4.5.13 
libchromiumcontent: 62.0.3202.75 
V8: 6.2.414.36 
Node.js: 7.9.0 
Update Channel: Release 
OS Platform: Microsoft Windows 
OS Release: 10.0.16299 
OS Architecture: x64

Reproducible on current live release:
Yes

Additional Information

Similar to #8463.

@jonathansampson jonathansampson added bug webcompat needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. labels Nov 4, 2017
@jonathansampson
Copy link
Collaborator Author

I'm working on a repro of this, and will be speaking with a community user against Monday (Nov 04, 2017) to get access to the original, impacted website (requires credentials, which I do not have at this time).

@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@hugobuddel
Copy link
Contributor

"Save link as" does not work at all with websites that require authentication. It will download the login page, I think because it does not send cookies with credentials, see #11444. So maybe that's what you are seeing? Or did you manage to reproduce this on a authentication-free website?

@bsclifton bsclifton added the stale label Sep 9, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Sep 9, 2018
@bsclifton bsclifton removed the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Sep 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants