`update.sh` updates all versions for non-matching version input

Open Beginner friendly
#2,436 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
64/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Quiet
Tech stack
docker, shell

Research direction

Start by reading update.sh, focusing on how the -s MAJOR_VERSION argument is parsed and matched against available versions. Run ./update.sh -s 26 to reproduce the current behavior, then verify that unsupported versions such as 23 or 26 fail with an error while supported versions continue updating normally.

Written by the indexing model from the issue text.

Description

build process

Environment

  • Platform: Ubuntu 24.04.4 LTS
  • Docker Version: N/A
  • Node.js Version: 24.14.1
  • Image Tag: N/A

Expected Behavior

If the update.sh script is run using a MAJOR_VERSION that is not set up, such as 23 (EOL) or 26 (not yet released), the script should fail with an error message.

Current Behavior

The script acts as though no MAJOR_VERSION was specified and updates all available versions.

Possible Solution

Steps to Reproduce

Execute:

./update.sh -s 26

Additional Information

Logs
$ ./update.sh -s 26
Updating version 20...
Updating version 22...
Updating version 24...
Updating version 25...
20/bookworm-slim/Dockerfile is already up to date!
20/bookworm/Dockerfile is already up to date!
20/bullseye-slim/Dockerfile is already up to date!
20/bullseye/Dockerfile is already up to date!
20/trixie-slim/Dockerfile is already up to date!
20/trixie/Dockerfile is already up to date!
22/trixie/Dockerfile is already up to date!
22/bookworm/Dockerfile is already up to date!
22/trixie-slim/Dockerfile is already up to date!
22/bookworm-slim/Dockerfile is already up to date!
22/bullseye/Dockerfile is already up to date!
22/bullseye-slim/Dockerfile is already up to date!
24/bullseye-slim/Dockerfile is already up to date!
24/bookworm-slim/Dockerfile is already up to date!
24/bookworm/Dockerfile is already up to date!
25/bullseye/Dockerfile is already up to date!
25/bookworm/Dockerfile is already up to date!
24/bullseye/Dockerfile is already up to date!
24/trixie-slim/Dockerfile is already up to date!
24/trixie/Dockerfile is already up to date!
25/bookworm-slim/Dockerfile is already up to date!
25/trixie/Dockerfile is already up to date!
25/bullseye-slim/Dockerfile is already up to date!
25/trixie-slim/Dockerfile is already up to date!
20/alpine3.22/Dockerfile is already up to date!
22/alpine3.22/Dockerfile is already up to date!
22/alpine3.23/Dockerfile is already up to date!
20/alpine3.23/Dockerfile is already up to date!
25/alpine3.23/Dockerfile is already up to date!
24/alpine3.22/Dockerfile is already up to date!
24/alpine3.23/Dockerfile is already up to date!
25/alpine3.22/Dockerfile is already up to date!
Done!
Dominant language
Dockerfile
Stars
8.6k
Forks
2k
Avg merge
10h 19m
Merged PRs (30d)
16

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from nodejs/docker-node

All issues in nodejs/docker-node

Similar issues

More Build System issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.