hydroframe / SandTank

Sand Tank Outreach Tool
https://hydroframe.github.io/SandTank/
Other
4 stars 5 forks source link

Node.js version dependency documentation // JOSE REVIEW #64

Closed bjrubinstein closed 1 year ago

bjrubinstein commented 1 year ago

@reedmaxwell The instructions for building the web client in the readme or running the development web server in the Web Application page of the documentation have the following commands respectively:

cd ./client
npm install
npm run build
cd client
npm run serve

These fail with the a ERR_OSSL_EVP_UNSUPPORTED on Node version 18 or higher. Last time the documentation was updated version 18 hadn't been released but now that the dependency exists could you please update the documentation to reflected it?

reedmaxwell commented 1 year ago

thanks, tagging @jourdain

jourdain commented 1 year ago

A quick google search lead me to export NODE_OPTIONS=--openssl-legacy-provider. Not sure if that fix it or not.

bjrubinstein commented 1 year ago

@jourdain https://github.com/jourdain Thanks for the reply. I tried that but wasn't able to get it to work. I think that might be because I'm on Windows (the command was set, not export). I found it was easier to just switch to the LTS release of Node rather than try to troubleshoot it further.

On Mon, Aug 8, 2022 at 11:46 AM Sebastien Jourdain @.***> wrote:

A quick google search lead me to export NODE_OPTIONS=--openssl-legacy-provider https://stackoverflow.com/questions/69394632/webpack-build-failing-with-err-ossl-evp-unsupported. Not sure if that fix it or not.

— Reply to this email directly, view it on GitHub https://github.com/hydroframe/SandTank/issues/64#issuecomment-1208361899, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYPRYSVCHYAOPDZ6GNBBDB3VYE2VHANCNFSM543D3BAQ . You are receiving this because you authored the thread.Message ID: @.***>

lecondon commented 1 year ago

I added a note to the readme that version 17 or lower is required. I think that should address this issue.

bjrubinstein commented 1 year ago

Thanks @lecondon . That looks good to me.