-
Notifications
You must be signed in to change notification settings - Fork 55
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
chore: update Dockerfile to latest debian, libssl & node 20 #73
Conversation
TheGreatRefrigerator
commented
Nov 14, 2023
- fixes issues with libssl encountered when updating to latest vroom-docker
Dockerfile
Outdated
@@ -27,15 +27,15 @@ RUN echo "Cloning and installing vroom-express release ${VROOM_EXPRESS_RELEASE}. | |||
git clone --branch $VROOM_EXPRESS_RELEASE https://github.com/VROOM-Project/vroom-express.git && \ | |||
cd vroom-express | |||
|
|||
FROM node:12-bullseye-slim as runstage | |||
FROM node:16.20-bookworm-slim as runstage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we're updating, we should update to LTS versions. bookworm is fine, but can we do node 20? 16 is EOL since a few months. (Unless 16.20 mean smth else than I think it does..)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, should be possible.(Nope i think it's node 16 😬 ) Still have to set up the full stack locally to test this properly. But that's a task for tomorrow :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. Tested some ors request with node 20 image that worked.
e902137
to
abc276f
Compare
- fixes issues with libssl encountered when updating to latest vroom-docker
abc276f
to
3f63487
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks LGTM 🚢