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

Upgrade to latest version #2

Open
wants to merge 82 commits into
base: master
Choose a base branch
from
Open

Upgrade to latest version #2

wants to merge 82 commits into from

Conversation

nprail
Copy link

@nprail nprail commented Dec 6, 2023

No description provided.

kjagiello and others added 30 commits April 18, 2019 08:23
The motivation behind the switch from `pdf2json` to `pdf-parse` was that the
former was not able to correctly read the PDF:s generated by the updated
Chromium version. Looking at the GitHub issues of that project showed that it
was quite buggy and not really maintained anymore. `pdf-parse` on the other
hand had no problem with PDF parsing, so the choice was easy.
Depending on the OS, Skia seems to render spaces and hyphens. When
running tests it introduces some incosistencies, so we normalize both
non-breaking spaces and soft hyphens to respective regular spaces and hyphens.
This came about because after running npm install we see the following:

found 39 high severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details
The previous image is no longer maintained also it does not even point to the correct source!
See this issue: microbox/node-url-to-pdf-api#4
We have built a new image that we will maintain also in the future as part for Restorecommerce.
…rabilities

Fixes lodash vulnerabilities by updating to latest revision
add option to create a browser by connecting to a chrome websocket endpoint
Point to different docker Image that is actually maintained
Adding browser executable path option
@nprail nprail had a problem deploying to eventive-pdf-service-staging December 18, 2023 19:32 Failure
@nprail nprail had a problem deploying to eventive-pdf-service-staging December 18, 2023 19:33 Failure
@nprail nprail temporarily deployed to eventive-pdf-service-staging December 18, 2023 19:38 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.