Skip to content

Adjust pdfjs-dist to reflect actual version being used via CDN.#1099

Merged
demiankatz merged 3 commits into
devfrom
revert-1098-dependabot/npm_and_yarn/dev/pdfjs-dist-4.6.82
Sep 2, 2024
Merged

Adjust pdfjs-dist to reflect actual version being used via CDN.#1099
demiankatz merged 3 commits into
devfrom
revert-1098-dependabot/npm_and_yarn/dev/pdfjs-dist-4.6.82

Conversation

@Saira-A

@Saira-A Saira-A commented Sep 2, 2024

Copy link
Copy Markdown
Contributor

Reverts #1098

@vercel

vercel Bot commented Sep 2, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
universalviewer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 2, 2024 2:31pm

Comment thread package.json Outdated
"mediaelement-plugins": "2.5.1",
"openseadragon": "2.4.2",
"pdfjs-dist": "4.6.82",
"pdfjs-dist": "2.0.161",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think it's worth changing this to 3.11.174 while we're changing it again anyway, so it at least reflects the version that is actually being used? That way, if we get any security alerts, we'll actually know they're for real. :-)

@demiankatz demiankatz changed the title Revert "Bump pdfjs-dist from 2.0.161 to 4.6.82" Adjust pdfjs-dist to reflect actual version being used via CDN. Sep 2, 2024

@demiankatz demiankatz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Saira-A -- I've renamed the PR to reflect what's actually happening here now, and will merge it.

@demiankatz demiankatz merged commit c5268da into dev Sep 2, 2024
@demiankatz demiankatz deleted the revert-1098-dependabot/npm_and_yarn/dev/pdfjs-dist-4.6.82 branch September 2, 2024 14:34
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.

2 participants