Merge pull request #147 from brendandburns/fixes

Fixes to be more lenient on configs without users.

Author: k8s-ci-robot

src/config.ts

@@ -119,7 +119,7 @@ export class KubeConfig {
             opts.strictSSL = false;
         }
 
-        if (user.username) {
+        if (user && user.username) {
             opts.auth = {
                 password: user.password,
                 username: user.username,
@@ -253,7 +253,9 @@ export class KubeConfig {
     private applyHTTPSOptions(opts: request.Options | https.RequestOptions) {
         const cluster = this.getCurrentCluster();
         const user = this.getCurrentUser();
-
+        if (!user) {
+            return;
+        }
         const ca = cluster != null ? this.bufferFromFileOrString(cluster.caFile, cluster.caData) : null;
         if (ca) {
             opts.ca = ca;
@@ -270,6 +272,9 @@ export class KubeConfig {
 
     private applyAuthorizationHeader(opts: request.Options | https.RequestOptions) {
         const user = this.getCurrentUser();
+        if (!user) {
+            return;
+        }
         let token: string | null = null;
 
         if (user.authProvider && user.authProvider.config) {

src/config_test.ts

@@ -355,25 +355,6 @@ describe('KubeConfig', () => {
                 ]);
             }).to.throw('contexts[1].context is missing');
         });
-        it('should fail if user is missing from context', () => {
-            expect(() => {
-                newContexts([
-                    {
-                        name: 'some-cluster',
-                        context: {
-                            cluster: 'foo',
-                            user: 'bar',
-                        },
-                    },
-                    {
-                        name: 'bar',
-                        context: {
-                            cluster: 'baz',
-                        },
-                    },
-                ]);
-            }).to.throw('contexts[1].context.user is missing');
-        });
         it('should fail if context is missing from context', () => {
             expect(() => {
                 newContexts([

src/config_types.ts

@@ -101,13 +101,10 @@ function contextIterator(): u.ListIterator<any, Context> {
         if (!elt.context.cluster) {
             throw new Error(`contexts[${i}].context.cluster is missing`);
         }
-        if (!elt.context.user) {
-            throw new Error(`contexts[${i}].context.user is missing`);
-        }
         return {
             cluster: elt.context.cluster,
             name: elt.name,
-            user: elt.context.user,
+            user: (elt.context.user ? elt.context.user : undefined),
         };
     };
 }