-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Upgrade version of Node. Update dependencies #44
Conversation
@blcham |
@palagdan Try to do testing PR from main (e.g. modify README.md only) and see if docker build works ... See also warning here: |
These warnings were due to the actions/checkout@v2 in workflow file which had be deprecated. I updated to actions/checkout@v3. Now it should work fine. |
@palagdan I tried it locally and I get the same error:
|
I executed it as follows:
|
@palagdan do i understand it correcly, that you cannot simulate failure of docker build ? |
Locally no. |
The problem was due to the updated Node.js version. Starting with Node.js v17, the default OpenSSL configuration changed, and some older cryptographic algorithms and modes are no longer supported by default. That’s why the solution I found is to set the environment variable: |
@palagdan what would it require to remove this option, can we even update it more so we would not need |
…new version of webpack
@palagdan, moreover, did you test that |
I updated webpack to the latest version and made changes to the configuration because of the following message from the webpack team:
I decided to disable this polyfill for now. |
Yes, after each update |
Yes, you’re right. I updated webpack to the latest version and it is no longer needed. |
Resolve #43