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 #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
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 #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