Skip to content

Conversation

@gadamo
Copy link

@gadamo gadamo commented Dec 22, 2022

Fix image build error. While there, also updated zerotier to latest minor version.

Pull Request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • [X ] Build-related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Base image for building the frontend is referenced with its tag on
the Dockerfile (FROM node:lts-alpine). The tag has been recently
changed, and now points to version 18 of nodejs.

Attempting to build with new lts (v.18), results in an error when
building the frontend:

[frontend-build 8/8] RUN yarn build:
#0 3.553 Creating an optimized production build...
#0 4.139 Error: error:0308010C:digital envelope routines::unsupported

Issue Number: N/A

What is the new behavior?

Adding the ENV var has fixed the build issue, and the container image
is now produced without errors.

Does this introduce a breaking change?

  • Yes
  • [X ] No

George Adamopoulos added 2 commits December 22, 2022 22:23
Base image for building the frontend is referenced with its tag on
the Dockerfile (FROM node:lts-alpine). The tag has been recently
changed, and now points to version 18 of nodejs.

Attempting to build with new lts (v.18), results in an error when
building the frontend:
 > [frontend-build 8/8] RUN yarn build:

Adding the ENV var has fixed the build issue, and the container image
is now produced without errors.
@gadamo gadamo requested a review from dec0dOS as a code owner December 22, 2022 20:45
@dec0dOS dec0dOS merged commit fbd10cb into dec0dOS:main Dec 28, 2022
@dec0dOS
Copy link
Owner

dec0dOS commented Dec 28, 2022

Hello, @gadamo!

Thanks for your PR!

@github-actions github-actions bot locked and limited conversation to collaborators Dec 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants