Merge pull request #1022 from davidgamero/davidgamero/request-migrati…

…on-auth

remove request usages from auth & config

src/auth.ts

@@ -1,9 +1,8 @@
 import https = require('https');
-import request = require('request');
 
 import { User } from './config_types';
 
 export interface Authenticator {
     isAuthProvider(user: User): boolean;
-    applyAuthentication(user: User, opts: request.Options | https.RequestOptions): Promise<void>;
+    applyAuthentication(user: User, opts: https.RequestOptions): Promise<void>;
 }

src/config.ts

@@ -5,7 +5,6 @@ import yaml = require('js-yaml');
 import net = require('net');
 import path = require('path');
 
-import request = require('request');
 import shelljs = require('shelljs');
 
 import * as api from './api';
@@ -459,7 +458,7 @@ export class KubeConfig implements SecurityAuthentication {
         return this.getContextObject(this.currentContext);
     }
 
-    private applyHTTPSOptions(opts: request.Options | https.RequestOptions): void {
+    private applyHTTPSOptions(opts: https.RequestOptions): void {
         const cluster = this.getCurrentCluster();
         const user = this.getCurrentUser();
         if (!user) {
@@ -483,7 +482,7 @@ export class KubeConfig implements SecurityAuthentication {
         }
     }
 
-    private async applyAuthorizationHeader(opts: request.Options | https.RequestOptions): Promise<void> {
+    private async applyAuthorizationHeader(opts: https.RequestOptions): Promise<void> {
         const user = this.getCurrentUser();
         if (!user) {
             return;
@@ -504,7 +503,7 @@ export class KubeConfig implements SecurityAuthentication {
         }
     }
 
-    private async applyOptions(opts: request.Options | https.RequestOptions): Promise<void> {
+    private async applyOptions(opts: https.RequestOptions): Promise<void> {
         this.applyHTTPSOptions(opts);
         await this.applyAuthorizationHeader(opts);
     }