-
Notifications
You must be signed in to change notification settings - Fork 11
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
[CLI-2340] Remove .netrc support #2751
Conversation
Let's explain that they'll be read from the configuration file or macOS keychain so users don't get worried the |
@@ -40,7 +40,6 @@ require ( | |||
github.com/confluentinc/ccloud-sdk-go-v2/stream-designer v0.3.0 | |||
github.com/confluentinc/confluent-kafka-go v1.9.3-RC3 | |||
github.com/confluentinc/go-editor v0.11.0 | |||
github.com/confluentinc/go-netrc v0.0.0-20220321173724-4d50f36ff450 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's archive https://github.com/confluentinc/go-netrc after the v4 release?
Co-authored-by: Brian Strauch <bstrauch@confluent.io>
Co-authored-by: Brian Strauch <bstrauch@confluent.io>
pkg/config/context.go
Outdated
@@ -27,6 +27,7 @@ type Context struct { | |||
FeatureFlags *FeatureFlags `json:"feature_flags,omitempty"` | |||
|
|||
// Deprecated | |||
NetrcMachineName string `json:"netrc_machine_name"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason for keeping this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So that I can read it in Load()
and move it to the new machine_name
field for users upgrading from 3.x. I believe we still use it for config credentials.
I think we had Windows support for this too... maybe we can drop the part about "Unix systems" just in case. |
Release Notes
Breaking Changes
.netrc
fileNew Features
Bug Fixes
Checklist
What
.netrc support for saved credentials was deprecated a while ago. This PR removes it.
References
Test & Review
Manually tested config migration: ran a command w/ this version and checked that
netrc_machine_name
was replaced bymachine_name
in the config file.