address review comments

Signed-off-by: Jim Ehrismann <jehrismann@mirantis.com>
This commit is contained in:
Jim Ehrismann committed 2021-10-22 17:38:57 -04:00
1 parent 264045cd37
commit 005b993933
2 files changed
+4 -6

No files matched your search

+3 -5
View File
@@ -134,11 +134,9 @@ export class PortForwardRoute {
if (!portForward) { if (!portForward) {
logger.info(`Creating a new port-forward ${namespace}/${resourceType}/${resourceName}:${port}`); logger.info(`Creating a new port-forward ${namespace}/${resourceType}/${resourceName}:${port}`);
let thePort = 0; const thePort = 0 < forwardPort && forwardPort < 65536
? forwardPort
if (forwardPort > 0 && forwardPort < 65536) { : 0;
thePort = forwardPort;
}
portForward = new PortForward(await cluster.getProxyKubeconfigPath(), { portForward = new PortForward(await cluster.getProxyKubeconfigPath(), {
clusterId: cluster.id, clusterId: cluster.id,
@@ -61,7 +61,7 @@ export class PortForwardDialog extends Component<Props> {
static open(portForward: ForwardedPort, options: PortForwardDialogOpenOptions = { openInBrowser: false }) { static open(portForward: ForwardedPort, options: PortForwardDialogOpenOptions = { openInBrowser: false }) {
dialogState.isOpen = true; dialogState.isOpen = true;
dialogState.data = portForward; dialogState.data = portForward;
dialogState.useHttps = Boolean(portForward.protocol === "https"); dialogState.useHttps = portForward.protocol === "https";
dialogState.openInBrowser = options.openInBrowser; dialogState.openInBrowser = options.openInBrowser;
} }