Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDF viewer on public links doesn't work #25251

Closed
schiessle opened this issue Jan 21, 2021 · 4 comments
Closed

PDF viewer on public links doesn't work #25251

schiessle opened this issue Jan 21, 2021 · 4 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug

Comments

@schiessle
Copy link
Member

Tested with c.nc.c (Nextcloud 21 beta 6)

When I share a PDF as public link it doesn't open the PDF reader. For large files I only see the download button:

image

For smaller PDFs I get a preview but only from the first page.

I would expect that the PDF Viewer opens and I can read the whole document in the browser

@schiessle schiessle added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 21, 2021
@megothub
Copy link

megothub commented Feb 5, 2021

+1
It works when you are logged in.
But, it doesn't work for public shares. Even on password protected shares

On my side Latest NC 20.0.7
Ubuntu 20.04.2

The only workaround is to disable PDF Viewer

here is the related ticket
nextcloud/files_pdfviewer#286

@a1466d44-d3dc-4c0b-90c7-315b088731d7

Same here
NC 20.0.07 on Ubuntu 20.04 (non docker)
Firefox on Ubuntu 20.04

Warning: getHexString - ignoring invalid character: 33 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 79 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 84 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 89 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 80 pdf.worker.js:1705:13
Warning: getHexString - ignoring additional invalid characters. pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 104 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 116 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 109 pdf.worker.js:1705:13
Warning: getHexString - ignoring invalid character: 108

Error in app.js:817:16
_callee7$/</< app.js:817
promise callback*_callee7$/< app.js:815
promise callback*_callee7$ app.js:776
Babel 8
webViewerOpenFileViaURL app.js:1947
webViewerInitialized app.js:1917
promise callbackrun app.js:478
webViewerLoad viewer.js:218
EventListener.handleEvent
viewer.js:228
Webpack 3

No Problem when you are logged in.
Test can be done, by sharing a folder, and opening that link in a Private Browsing session.

@a1466d44-d3dc-4c0b-90c7-315b088731d7

Issue can be closed.
Was fixed in 20.0.8

https://nextcloud.com/changelog/#20-0-8 (Fix opening pdf files in public folder shares (files_pdfviewer#321))
nextcloud/files_pdfviewer#321

Sorry for the inconvenience

@szaimen
Copy link
Contributor

szaimen commented Jun 25, 2021

there was a new pdf issue on NC21.0.2 but it will be fixed with NC21.0.3

@szaimen szaimen closed this as completed Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug
Projects
None yet
Development

No branches or pull requests

4 participants