Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Go 1.18 #816

Merged
merged 2 commits into from
Jul 27, 2022
Merged

Upgrade to Go 1.18 #816

merged 2 commits into from
Jul 27, 2022

Conversation

phillebaba
Copy link
Member

@phillebaba phillebaba commented Jul 5, 2022

Upgrade to use Go 1.18. This is required to unblock #786.

The major issues which seem like they could affect end users are security and crypto related. This information should be published even if it may not affect many users.

  • TLS now defaults to 1.2 for client connections
  • crypto/x509 rejects SHA-1 hash functions

This PR requires pkg to be updated first.

@hiddeco
Copy link
Member

hiddeco commented Jul 5, 2022

xref: fluxcd/flux2#2696

@stefanprodan stefanprodan added the area/ci CI related issues and pull requests label Jul 27, 2022
@stefanprodan stefanprodan changed the title Upgrade to go 1.18 Upgrade to Go 1.18 Jul 27, 2022
@stefanprodan stefanprodan marked this pull request as ready for review July 27, 2022 16:01
phillebaba and others added 2 commits July 27, 2022 19:02
Signed-off-by: Philip Laine <philip.laine@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan merged commit d9dfb9d into main Jul 27, 2022
@stefanprodan stefanprodan deleted the go-1-18 branch July 27, 2022 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants