I’m passing on an issue report from our client - I erroneously didn’t have pdfJS express pinned to a particular version in our installation and production has automatically updated to 8.4. We’ll roll it back to 8.3 for now but here’s the report:
Which product are you using?
PDF.js Express Viewer
PDF.js Express Version
8.4
Detailed description of issue
“It is using a huge amount of memory on Safari on my dev machine, i.e. showing an alert every few minutes. and I’ve only seen those memory errors before on a 10 year old iPad mini.”
Expected behaviour
It will run on Safari without producing the alerts. I’ll ask Michelle for a screenshot of an alert.
Does your issue happen with every document, or just one?
I assume every document based on her report.
We will investigate the memory usage issues - there was no core functionality changes between 8.3 and 8.4 so its a bit surprising to see this.
The errors you are seeing when upgrading are probably related to not upgrading properly - please make sure to completely wipe out the static assets and re-copy them over from the new version.
I’ll get back to you with my findings on the memory usage.
Also just as a test, would you be able to ask Michelle if she can reproduce that error playing around with WebViewer? Try clicking around here a bit in Safari and see if you get the same alert.
I tried investigating but I could not reproduce the problem. I tested PDF.js Express Viewer 8.4 on Safari with many different documents (some as large as 10,000 pages) and I didn’t have any memory issues. I suspect this is either document specific or device specific.
I will need more details or a better way to reproduce in order to help at the moment. If you could isolate the problem to a document or a set of documents that would be very useful too.
I’m heading back from a conference in Portland, Oregon so might be a bit slow replying, but I’ll pass your reply on to Michelle and see if she can give you a reliable reproducible case.
I’m pretty sure the deployment process starts with just the package.json so don’t think there should be any stale files in there. I’ll double check and get back to you.