mirror of
https://github.com/kubernetes/client-go.git
synced 2025-06-25 14:41:53 +00:00
Merge pull request #88940 from tianhongw/master
fix golint errors in staging/src/k8s.io/client-go/plugin/pkg/client/a… Kubernetes-commit: 5bfe8fe2b9024f0319c053d5514b491f7c18aad8
This commit is contained in:
commit
1be5940d0d
2
Godeps/Godeps.json
generated
2
Godeps/Godeps.json
generated
@ -352,7 +352,7 @@
|
|||||||
},
|
},
|
||||||
{
|
{
|
||||||
"ImportPath": "k8s.io/apimachinery",
|
"ImportPath": "k8s.io/apimachinery",
|
||||||
"Rev": "ff54c5b023af"
|
"Rev": "e002472249f8"
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"ImportPath": "k8s.io/gengo",
|
"ImportPath": "k8s.io/gengo",
|
||||||
|
4
go.mod
4
go.mod
@ -29,7 +29,7 @@ require (
|
|||||||
golang.org/x/time v0.0.0-20191024005414-555d28b269f0
|
golang.org/x/time v0.0.0-20191024005414-555d28b269f0
|
||||||
google.golang.org/appengine v1.5.0 // indirect
|
google.golang.org/appengine v1.5.0 // indirect
|
||||||
k8s.io/api v0.0.0-20200410021914-5778e4f3d00d
|
k8s.io/api v0.0.0-20200410021914-5778e4f3d00d
|
||||||
k8s.io/apimachinery v0.0.0-20200410021338-ff54c5b023af
|
k8s.io/apimachinery v0.0.0-20200410061239-e002472249f8
|
||||||
k8s.io/klog v1.0.0
|
k8s.io/klog v1.0.0
|
||||||
k8s.io/utils v0.0.0-20200324210504-a9aa75ae1b89
|
k8s.io/utils v0.0.0-20200324210504-a9aa75ae1b89
|
||||||
sigs.k8s.io/yaml v1.2.0
|
sigs.k8s.io/yaml v1.2.0
|
||||||
@ -39,5 +39,5 @@ replace (
|
|||||||
golang.org/x/sys => golang.org/x/sys v0.0.0-20190813064441-fde4db37ae7a // pinned to release-branch.go1.13
|
golang.org/x/sys => golang.org/x/sys v0.0.0-20190813064441-fde4db37ae7a // pinned to release-branch.go1.13
|
||||||
golang.org/x/tools => golang.org/x/tools v0.0.0-20190821162956-65e3620a7ae7 // pinned to release-branch.go1.13
|
golang.org/x/tools => golang.org/x/tools v0.0.0-20190821162956-65e3620a7ae7 // pinned to release-branch.go1.13
|
||||||
k8s.io/api => k8s.io/api v0.0.0-20200410021914-5778e4f3d00d
|
k8s.io/api => k8s.io/api v0.0.0-20200410021914-5778e4f3d00d
|
||||||
k8s.io/apimachinery => k8s.io/apimachinery v0.0.0-20200410021338-ff54c5b023af
|
k8s.io/apimachinery => k8s.io/apimachinery v0.0.0-20200410061239-e002472249f8
|
||||||
)
|
)
|
||||||
|
2
go.sum
2
go.sum
@ -190,7 +190,7 @@ gopkg.in/yaml.v2 v2.2.8/go.mod h1:hI93XBmqTisBFMUTm0b8Fm+jr3Dg1NNxqwp+5A1VGuI=
|
|||||||
honnef.co/go/tools v0.0.0-20190102054323-c2f93a96b099/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4=
|
honnef.co/go/tools v0.0.0-20190102054323-c2f93a96b099/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4=
|
||||||
honnef.co/go/tools v0.0.0-20190106161140-3f1c8253044a/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4=
|
honnef.co/go/tools v0.0.0-20190106161140-3f1c8253044a/go.mod h1:rf3lG4BRIbNafJWhAfAdb/ePZxsR/4RtNHQocxwk9r4=
|
||||||
k8s.io/api v0.0.0-20200410021914-5778e4f3d00d/go.mod h1:kNZHsfUjF+wX4CfMBMSHs4FliofoB7PN0ILSMBbxyYc=
|
k8s.io/api v0.0.0-20200410021914-5778e4f3d00d/go.mod h1:kNZHsfUjF+wX4CfMBMSHs4FliofoB7PN0ILSMBbxyYc=
|
||||||
k8s.io/apimachinery v0.0.0-20200410021338-ff54c5b023af/go.mod h1:imoz42hIYwpLTRWXU8pdJ9IE8DbxUsnU9lyVN8Y1SNo=
|
k8s.io/apimachinery v0.0.0-20200410061239-e002472249f8/go.mod h1:imoz42hIYwpLTRWXU8pdJ9IE8DbxUsnU9lyVN8Y1SNo=
|
||||||
k8s.io/gengo v0.0.0-20190128074634-0689ccc1d7d6/go.mod h1:ezvh/TsK7cY6rbqRK0oQQ8IAqLxYwwyPxAX1Pzy0ii0=
|
k8s.io/gengo v0.0.0-20190128074634-0689ccc1d7d6/go.mod h1:ezvh/TsK7cY6rbqRK0oQQ8IAqLxYwwyPxAX1Pzy0ii0=
|
||||||
k8s.io/klog v0.0.0-20181102134211-b9b56d5dfc92/go.mod h1:Gq+BEi5rUBO/HRz0bTSXDUcqjScdoY3a9IHpCEIOOfk=
|
k8s.io/klog v0.0.0-20181102134211-b9b56d5dfc92/go.mod h1:Gq+BEi5rUBO/HRz0bTSXDUcqjScdoY3a9IHpCEIOOfk=
|
||||||
k8s.io/klog v0.3.0/go.mod h1:Gq+BEi5rUBO/HRz0bTSXDUcqjScdoY3a9IHpCEIOOfk=
|
k8s.io/klog v0.3.0/go.mod h1:Gq+BEi5rUBO/HRz0bTSXDUcqjScdoY3a9IHpCEIOOfk=
|
||||||
|
@ -35,7 +35,7 @@ import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
const (
|
const (
|
||||||
cfgIssuerUrl = "idp-issuer-url"
|
cfgIssuerURL = "idp-issuer-url"
|
||||||
cfgClientID = "client-id"
|
cfgClientID = "client-id"
|
||||||
cfgClientSecret = "client-secret"
|
cfgClientSecret = "client-secret"
|
||||||
cfgCertificateAuthority = "idp-certificate-authority"
|
cfgCertificateAuthority = "idp-certificate-authority"
|
||||||
@ -109,9 +109,9 @@ func (c *clientCache) setClient(clusterAddress, issuer, clientID string, client
|
|||||||
}
|
}
|
||||||
|
|
||||||
func newOIDCAuthProvider(clusterAddress string, cfg map[string]string, persister restclient.AuthProviderConfigPersister) (restclient.AuthProvider, error) {
|
func newOIDCAuthProvider(clusterAddress string, cfg map[string]string, persister restclient.AuthProviderConfigPersister) (restclient.AuthProvider, error) {
|
||||||
issuer := cfg[cfgIssuerUrl]
|
issuer := cfg[cfgIssuerURL]
|
||||||
if issuer == "" {
|
if issuer == "" {
|
||||||
return nil, fmt.Errorf("Must provide %s", cfgIssuerUrl)
|
return nil, fmt.Errorf("Must provide %s", cfgIssuerURL)
|
||||||
}
|
}
|
||||||
|
|
||||||
clientID := cfg[cfgClientID]
|
clientID := cfg[cfgClientID]
|
||||||
@ -216,7 +216,7 @@ func (r *roundTripper) RoundTrip(req *http.Request) (*http.Response, error) {
|
|||||||
return r.wrapped.RoundTrip(r2)
|
return r.wrapped.RoundTrip(r2)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (t *roundTripper) WrappedRoundTripper() http.RoundTripper { return t.wrapped }
|
func (r *roundTripper) WrappedRoundTripper() http.RoundTripper { return r.wrapped }
|
||||||
|
|
||||||
func (p *oidcAuthProvider) idToken() (string, error) {
|
func (p *oidcAuthProvider) idToken() (string, error) {
|
||||||
p.mu.Lock()
|
p.mu.Lock()
|
||||||
@ -240,7 +240,7 @@ func (p *oidcAuthProvider) idToken() (string, error) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Determine provider's OAuth2 token endpoint.
|
// Determine provider's OAuth2 token endpoint.
|
||||||
tokenURL, err := tokenEndpoint(p.client, p.cfg[cfgIssuerUrl])
|
tokenURL, err := tokenEndpoint(p.client, p.cfg[cfgIssuerURL])
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return "", err
|
return "", err
|
||||||
}
|
}
|
||||||
@ -263,7 +263,7 @@ func (p *oidcAuthProvider) idToken() (string, error) {
|
|||||||
// providers (Okta) don't return this value.
|
// providers (Okta) don't return this value.
|
||||||
//
|
//
|
||||||
// See https://github.com/kubernetes/kubernetes/issues/36847
|
// See https://github.com/kubernetes/kubernetes/issues/36847
|
||||||
return "", fmt.Errorf("token response did not contain an id_token, either the scope \"openid\" wasn't requested upon login, or the provider doesn't support id_tokens as part of the refresh response.")
|
return "", fmt.Errorf("token response did not contain an id_token, either the scope \"openid\" wasn't requested upon login, or the provider doesn't support id_tokens as part of the refresh response")
|
||||||
}
|
}
|
||||||
|
|
||||||
// Create a new config to persist.
|
// Create a new config to persist.
|
||||||
|
Loading…
Reference in New Issue
Block a user