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

insecure: remove experimental notice #5069

Merged
merged 1 commit into from Dec 22, 2021

Conversation

adambabik
Copy link
Contributor

@adambabik adambabik commented Dec 20, 2021

In #4718, the insecure package was used to implement WithInsecure() among others and it was suggested that insecure.NewCredentials() should be used instead. However, the insecure package still contains a notice that its status is experimental. I believe that notice can be deleted now.

RELEASE NOTES:

  • credentials/insecure: remove experimental status of package

@linux-foundation-easycla
Copy link

@linux-foundation-easycla linux-foundation-easycla bot commented Dec 20, 2021

CLA Signed

The committers are authorized under a signed CLA.

@dfawley dfawley added this to the 1.44 Release milestone Dec 22, 2021
Copy link
Contributor

@dfawley dfawley left a comment

It's been a few months, so this is probably fine... The API is pretty narrow anyway.

@easwars easwars merged commit db9fdf7 into grpc:master Dec 22, 2021
10 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants