From b58eb099f7d349fe7980e98b22c0bca347ba6c51 Mon Sep 17 00:00:00 2001 From: Travis Truman Date: Fri, 16 Apr 2021 08:44:04 -0400 Subject: [PATCH] Responding to review feedback Signed-off-by: Travis Truman --- docs/features/kubernetes/configuration.md | 2 +- .../src/cluster-locator/ConfigClusterLocator.ts | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/docs/features/kubernetes/configuration.md b/docs/features/kubernetes/configuration.md index b96b1461de..8222fbb400 100644 --- a/docs/features/kubernetes/configuration.md +++ b/docs/features/kubernetes/configuration.md @@ -82,7 +82,7 @@ cluster. Valid values are: ##### `clusters.\*.skipTLSVerify` This determines whether or not the Kubernetes client verifies the TLS -certificate presented by the API server. +certificate presented by the API server. Defaults to `false`. ##### `clusters.\*.serviceAccountToken` (optional) diff --git a/plugins/kubernetes-backend/src/cluster-locator/ConfigClusterLocator.ts b/plugins/kubernetes-backend/src/cluster-locator/ConfigClusterLocator.ts index 62b127cd60..169e50534f 100644 --- a/plugins/kubernetes-backend/src/cluster-locator/ConfigClusterLocator.ts +++ b/plugins/kubernetes-backend/src/cluster-locator/ConfigClusterLocator.ts @@ -33,10 +33,7 @@ export class ConfigClusterLocator implements KubernetesClustersSupplier { name: c.getString('name'), url: c.getString('url'), serviceAccountToken: c.getOptionalString('serviceAccountToken'), - skipTLSVerify: - c.getOptionalBoolean('skipTLSVerify') === undefined - ? false - : c.getOptionalBoolean('skipTLSVerify'), + skipTLSVerify: c.getOptionalBoolean('skipTLSVerify') ?? false, authProvider: c.getString('authProvider'), }; }),