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

Update gRPC #167

Merged
merged 7 commits into from
Oct 18, 2023
Merged

Update gRPC #167

merged 7 commits into from
Oct 18, 2023

Conversation

subpop
Copy link
Collaborator

@subpop subpop commented Oct 16, 2023

Update gRPC versions to 1.58.3. The update to grpc 1.58.3 requires updating the minimum Go version to 1.19. Updating the Go version to 1.19 requires removing the usage of the deprecated ioutils package, replacing the golang.org/x/crypto/ssh/terminal package with golang.org/x/term, and replacing grpc.WithInsecure() with grpc.WIthTransportCredentials(insecure.NewCredentials()). This PR includes commits that fulfill those requirements.

Signed-off-by: Link Dupont <link@sub-pop.net>
Signed-off-by: Link Dupont <link@sub-pop.net>
@subpop subpop requested a review from jirihnidek October 16, 2023 14:40
Signed-off-by: Link Dupont <link@sub-pop.net>
Signed-off-by: Link Dupont <link@sub-pop.net>
@subpop subpop marked this pull request as draft October 16, 2023 15:32
Signed-off-by: Link Dupont <link@sub-pop.net>
grpc.WithInsecure() is deprecated in favor of
grpc.WithTransportCredentials().

Signed-off-by: Link Dupont <link@sub-pop.net>
Signed-off-by: Link Dupont <link@sub-pop.net>
@subpop subpop marked this pull request as ready for review October 16, 2023 16:12
@ehelms
Copy link
Member

ehelms commented Oct 16, 2023

Thanks for this @subpop ! From a high level overview the changes look good.

Copy link

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks to be in line with the changes we did elsewhere, +1

Copy link
Contributor

@jirihnidek jirihnidek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jirihnidek jirihnidek merged commit 791bb70 into yggdrasil-0.2 Oct 18, 2023
6 checks passed
@jirihnidek jirihnidek deleted the 0.2/update-grpc branch October 18, 2023 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants