Skip to content

Revert "update to go1.21.4" due to regressions / breaking changes.#4649

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:revert_update_golang_1.21.4
Nov 9, 2023
Merged

Revert "update to go1.21.4" due to regressions / breaking changes.#4649
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:revert_update_golang_1.21.4

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Unfortunately, the go1.21.4 security update exposed some regressions / breaking changes in moby (docker engine) and containerd. These issues are looked into, but in the meantime we should revert this patch.

This temporarily reintroduces CVE-2023-45284 and CVE-2023-45283.

This reverts commit 6472dab.

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Unfortunately, the go1.21.4 security update exposed some regressions / breaking
changes in moby (docker engine) and containerd. These issues are looked into,
but in the meantime we should revert this patch.

This temporarily reintroduces CVE-2023-45284 and CVE-2023-45283.

This reverts commit 6472dab.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 9, 2023

Codecov Report

Merging #4649 (4cf1c50) into master (ed1c8c1) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4649   +/-   ##
=======================================
  Coverage   59.74%   59.74%           
=======================================
  Files         288      288           
  Lines       24849    24849           
=======================================
  Hits        14846    14846           
  Misses       9117     9117           
  Partials      886      886           

@thaJeztah thaJeztah merged commit b90a3d4 into docker:master Nov 9, 2023
@thaJeztah thaJeztah deleted the revert_update_golang_1.21.4 branch November 9, 2023 12:04
@thaJeztah thaJeztah mentioned this pull request Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants