Merge pull request #596 from worgho2/fix/private-url-signing

NotionX · Jan 10, 2025 · 4985e0e · 4985e0e
2 parents 3991343 + 29111e5
commit 4985e0e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/packages/notion-client/src/notion-api.ts b/packages/notion-client/src/notion-api.ts
@@ -237,17 +237,17 @@ export class NotionAPI {
         // console.log(block, source)
 
         if (source) {
-          if (!source.includes('secure.notion-static.com')) {
-            return []
+          if (source.includes('secure.notion-static.com') || source.includes('prod-files-secure')) {
+            return {
+              permissionRecord: {
+                table: 'block',
+                id: block.id
+              },
+              url: source
+            };
           }
 
-          return {
-            permissionRecord: {
-              table: 'block',
-              id: block.id
-            },
-            url: source
-          }
+          return []
         }
       }
 

diff --git a/packages/react-notion-x/src/third-party/pdf.tsx b/packages/react-notion-x/src/third-party/pdf.tsx
@@ -2,7 +2,7 @@ import * as React from 'react'
 import { Document, Page, pdfjs } from 'react-pdf'
 
 // ensure pdfjs can find its worker script regardless of how react-notion-x is bundled
-pdfjs.GlobalWorkerOptions.workerSrc = `//unpkg.com/pdfjs-dist@${pdfjs.version}/legacy/build/pdf.worker.min.js`
+pdfjs.GlobalWorkerOptions.workerSrc = `//unpkg.com/pdfjs-dist@${pdfjs.version}/legacy/build/pdf.worker.min.mjs`
 
 export function Pdf({ file, ...rest }: { file: string }) {
   const [numPages, setNumPages] = React.useState<number>(0)