PDFs fail to load if behind basic auth

[bsclifton]

Carried over from brave/browser-laptop#8702

If a pdf file is behind basic auth, then it fails to display with our pdfjs extension.

Steps to reproduce:

1. Download this script and extract it to your home directory
   SimpleHTTPAuth.py.zip. Create a pdf file named type.pdf in the same directory.

2. Start it python SimpleHTTPAuth.py 9001 u:p

3. Navigate to http://localhost:9001/type.pdf

Note it works in Chrome with pdfjs but not in Brave built in.

CC @diracdeltas

[diracdeltas]

This is actually sort-of fixed in Brave core, except it prompts you to download the PDF after you authenticate. However if you try to access the PDF again once you've authenticated, then it displays the PDF in PDFJS.

[tpeacock19]

I think this may have been the wrong issue to identify; however I'm experiencing the following:

I have a kanboard instance here https://kb.treypeacock.com which you can login with u:test p:testpass

If you access the only shared project

select the task

attempt to view the pdf you are thrown the following error:

I get the same error when I need to login to my university's Active Directory and attempt to view a pdf.

[mwdiers]

Thank you for bringing this forward again. This is is the one issue that is preventing me from switching entirely to Brave, as I cannot view any reports in our ERP system without downloading them first (which is a pain in the rear when I am trying to pound through PO approvals for IT).

[diracdeltas]

@tpeacock19 that is a separate PDFJS issue (this one only applies to PDFs which are behind HTTP basic auth)

[diracdeltas]

confirmed that it works on Chrome with PDFJS extension

[tpeacock19]

@diracdeltas is there currently an open issue for this or should I make one?

[cndouglas]

@tpeacock19 See #1287

[diracdeltas]

actually i was mistaken, the basic auth issue is not fixed in Chrome either. what happens in Chrome is that it loads in pdfium (which i mistook for pdfjs). Brave disables pdfium so instead it tries to download the pdf.

[bbondy]

@tpeacock19 the one you ran into is not related to basic auth and is fixed here:
#1287
It'll be included in 0.57.x but I'll see if we can get it in 0.56.x which is about a week away.

[btlechowski]

Verification passed on

Brave	0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision	2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS	Windows 7 x64

Used STR from OP.

Verification passed on

Brave	0.58.12 Chromium: 71.0.3578.80 (Official Build) (64-bit)
Revision	2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS	Linux

Verified passed with

Brave	0.58.12 Chromium: 71.0.3578.80 (Official Build) (64-bit)
Revision	2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS	Mac OS X

• Verified steps from description