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.

Sign up for GitHub

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

blob-polyfill => fetch-blob and use /tmp for LocalStorage #424

Draft

jq170727 wants to merge 1 commit into smallbets:master from jq170727:blob-and-tmp

jq170727 commented Feb 28, 2023 •

edited

This request proposes to fix two small problems in userbase-js-node:

"blob-polyfill" no longer works, generating instead the following error:

% node index.js 
  xxx/node_modules/blob-polyfill/Blob.js:519
                            "-ms-scroll-limit" in document.documentElement.style &&
                                                  ^
  ReferenceError: document is not defined

On Vercel the current directory isn't writable by server-less functions.

To address these issues it:

replaces "blob-polyfill" with the newer "fetch-blob"
moves the parent of LocalStorage to "/tmp"

These changes work for my userbase use-case but would appreciate feedback and/or
suggestions as to alternative solutions.


blob-polyfill => fetch-blob and use /tmp for LocalStorage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment