RocketChat / RocketChat/Docker.Official.Image
docker image of RC 8.4 and 8.5 contains an old incompatible version of Deno
Nobody has claimed this yet.
- Dominant language
- Dockerfile
- Stars
- 337
- Forks
- 228
- Avg merge
- 3d 10h
- Merged PRs (30d)
- 5
Description
Docker image contains Deno 1.43.5, which is not the required version of Deno for Rocket.Chat 8.4 and 8.5 (it should be 2.3.1 - see https://github.com/RocketChat/Rocket.Chat/releases/tag/8.4.0)
We noticed the issue after recently upgrading from 7.13 to 8.5. The symptoms were that desktop notifications stopped working for users (at least using the desktop app, don't know about browser), apps would not allow to be enabled anymore and I think some other issues too, but can't remember right now. The mentioned ones were the most pressing.
Our solution was to replace Deno with 2.9.0 (latest) in our deployed image, but this should be fixed at the origin. Deno 2.3.1 also has a few vulnerabilities with a high score, but official images should include what's specified I think.
There is already a pull request fixing this in https://github.com/RocketChat/Docker.Official.Image/pull/286
I don't know if this fixes it properly, I see potential issues with RC <= 8.3 using the older version of Deno, but this HAS to be fixed sooner rather than later as the Docker image is severely broken as it is now.
I can look over that fix and change something if it's currently not fit for merging, please advise.
best regards,
Borut Mrak
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review pull request #286 in RocketChat/Docker.Official.Image first, then inspect how Deno is selected for RC 8.4 and 8.5 images and how older versions are handled. Done means the official images use the required Deno version without breaking compatibility for RC versions through 8.3.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- deno, dockerfile
- Domain
- devops, infrastructure
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100