RocketChat / RocketChat/Docker.Official.Image

docker image of RC 8.4 and 8.5 contains an old incompatible version of Deno

Open
#290 1 comment 2 reactions 0 assignees View on GitHub

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

  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.

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.