[PR #1170] [MERGED] feat: sealing robustness #1441

Closed
opened 2026-02-26 19:33:16 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1170
Author: @Mythie
Created: 5/23/2024
Status: Merged
Merged: 5/23/2024
Merged by: @Mythie

Base: mainHead: fix/sealing-robustness


📝 Commits (6)

  • 4c0b772 fix: rewrite form flattening handler
  • d58a881 fix: use noto sans for text insertion on pdfs
  • 3113284 fix: bump react-pdf and pdfjs-dist to handle cve
  • 64bec5f fix: remove console.log statements
  • 6e2363d Merge branch 'main' into fix/sealing-robustness
  • 1f3df51 fix: update font variable typo

📊 Changes

16 files changed (+211 additions, -56655 deletions)

View changed files

📝 apps/marketing/next.config.js (+5 -0)
📝 apps/marketing/public/pdf.worker.min.js (+1 -1)
📝 apps/web/next.config.js (+5 -0)
📝 apps/web/public/pdf.worker.min.js (+1 -56590)
📝 apps/web/src/app/(dashboard)/admin/documents/[id]/admin-actions.tsx (+7 -3)
📝 apps/web/src/app/(dashboard)/admin/documents/[id]/page.tsx (+1 -1)
📝 package-lock.json (+56 -52)
packages/assets/fonts/noto-sans.ttf (+0 -0)
📝 packages/lib/constants/pdf.ts (+1 -1)
📝 packages/lib/server-only/document/seal-document.ts (+2 -1)
packages/lib/server-only/pdf/flatten-form.ts (+112 -0)
📝 packages/lib/server-only/pdf/insert-field-in-pdf.ts (+6 -2)
📝 packages/trpc/server/admin-router/router.ts (+7 -1)
📝 packages/tsconfig/process-env.d.ts (+1 -0)
📝 packages/ui/package.json (+5 -3)
📝 turbo.json (+1 -0)

📄 Description

A series of changes to improve sealing robustness avoiding errors that have appeared during monitoring.

Additionally handles the recently published CVE affecting the pdfjs-dist library.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/1170 **Author:** [@Mythie](https://github.com/Mythie) **Created:** 5/23/2024 **Status:** ✅ Merged **Merged:** 5/23/2024 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/sealing-robustness` --- ### 📝 Commits (6) - [`4c0b772`](https://github.com/documenso/documenso/commit/4c0b772fc9698b3099ba4e16d7e594c39f2d4fb6) fix: rewrite form flattening handler - [`d58a881`](https://github.com/documenso/documenso/commit/d58a88196a97dfb4b3b3ba1b5cccfe56c663e117) fix: use noto sans for text insertion on pdfs - [`3113284`](https://github.com/documenso/documenso/commit/311328471e4e4ae567e1a8d917b0e06192baba05) fix: bump react-pdf and pdfjs-dist to handle cve - [`64bec5f`](https://github.com/documenso/documenso/commit/64bec5f29ceeca6e692a0381092157b500737589) fix: remove console.log statements - [`6e2363d`](https://github.com/documenso/documenso/commit/6e2363d48c87eb67573a1c168499531af51dfb47) Merge branch 'main' into fix/sealing-robustness - [`1f3df51`](https://github.com/documenso/documenso/commit/1f3df51371ae3226055be02d71ee1b16597e3a76) fix: update font variable typo ### 📊 Changes **16 files changed** (+211 additions, -56655 deletions) <details> <summary>View changed files</summary> 📝 `apps/marketing/next.config.js` (+5 -0) 📝 `apps/marketing/public/pdf.worker.min.js` (+1 -1) 📝 `apps/web/next.config.js` (+5 -0) 📝 `apps/web/public/pdf.worker.min.js` (+1 -56590) 📝 `apps/web/src/app/(dashboard)/admin/documents/[id]/admin-actions.tsx` (+7 -3) 📝 `apps/web/src/app/(dashboard)/admin/documents/[id]/page.tsx` (+1 -1) 📝 `package-lock.json` (+56 -52) ➕ `packages/assets/fonts/noto-sans.ttf` (+0 -0) 📝 `packages/lib/constants/pdf.ts` (+1 -1) 📝 `packages/lib/server-only/document/seal-document.ts` (+2 -1) ➕ `packages/lib/server-only/pdf/flatten-form.ts` (+112 -0) 📝 `packages/lib/server-only/pdf/insert-field-in-pdf.ts` (+6 -2) 📝 `packages/trpc/server/admin-router/router.ts` (+7 -1) 📝 `packages/tsconfig/process-env.d.ts` (+1 -0) 📝 `packages/ui/package.json` (+5 -3) 📝 `turbo.json` (+1 -0) </details> ### 📄 Description A series of changes to improve sealing robustness avoiding errors that have appeared during monitoring. Additionally handles the recently published CVE affecting the `pdfjs-dist` library. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 19:33:16 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/documenso#1441
No description provided.