Skip to content

Commit 3931f71

Browse files
mattfarinamattfarina
committed
Leverage the NodeJS.ErrnoException interface
Signed-off-by: Matt Farina <matt.farina@suse.com>
1 parent 8522d4c commit 3931f71

6 files changed

Lines changed: 23 additions & 23 deletions

File tree

‎src/k8s-engine/k3sHelper.ts‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import * as K8s from '@/k8s-engine/k8s';
2323
// TODO: Replace with the k8s version after kubernetes-client/javascript/pull/748 lands
2424
// const k8s = require('@kubernetes/client-node');
2525
import { findHomeDir } from '@/config/findHomeDir';
26-
import { isUnixError, isUnixFsError } from '@/typings/unix.interface';
26+
import { isUnixError } from '@/typings/unix.interface';
2727

2828
const console = Logging.k8s;
2929

@@ -103,7 +103,7 @@ export default class K3sHelper extends events.EventEmitter {
103103
}
104104
}
105105
} catch (ex) {
106-
if (isUnixFsError(ex) && ex.code !== 'ENOENT') {
106+
if ((ex as NodeJS.ErrnoException).code !== 'ENOENT') {
107107
throw ex;
108108
}
109109
}
@@ -382,7 +382,7 @@ export default class K3sHelper extends events.EventEmitter {
382382

383383
return (await Promise.all(promises)).filter(x => x)[0];
384384
} catch (ex) {
385-
if (isUnixFsError(ex) && ex.code !== 'ENOENT') {
385+
if ((ex as NodeJS.ErrnoException).code !== 'ENOENT') {
386386
throw ex;
387387
}
388388

@@ -536,7 +536,7 @@ export default class K3sHelper extends events.EventEmitter {
536536
return kubeConfigPath;
537537
}
538538
} catch (err) {
539-
if (isUnixFsError(err) && err.code !== 'ENOENT') {
539+
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
540540
throw err;
541541
}
542542
}
@@ -617,7 +617,7 @@ export default class K3sHelper extends events.EventEmitter {
617617
try {
618618
userConfig.loadFromFile(userPath, { onInvalidEntry: ActionOnInvalid.FILTER });
619619
} catch (err) {
620-
if (isUnixFsError(err) && err.code !== 'ENOENT') {
620+
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
621621
console.log(`Error trying to load kubernetes config file ${ userPath }:`, err);
622622
}
623623
// continue to merge into an empty userConfig == `{ contexts: [], clusters: [], users: [] }`

‎src/k8s-engine/unixlikeIntegrations.ts‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import paths from '@/utils/paths';
66
import resources from '@/resources';
77
import PathConflictManager from '@/main/pathConflictManager';
88
import * as window from '@/window';
9-
import { isUnixFsError } from '@/typings/unix.interface';
9+
import { isNodeError } from '@/typings/unix.interface';
1010

1111
const INTEGRATIONS = ['docker', 'helm', 'kubectl', 'nerdctl'];
1212
const console = Logging.background;
@@ -49,9 +49,9 @@ export default class UnixlikeIntegrations {
4949
this.#results[linkPath] = `Already linked to ${ currentDest }`;
5050
}
5151
} catch (error) {
52-
if (isUnixFsError(error) && error.code === 'ENOENT') {
52+
if ((error as NodeJS.ErrnoException).code === 'ENOENT') {
5353
this.#results[linkPath] = false;
54-
} else if (isUnixFsError(error) && error.code === 'EINVAL') {
54+
} else if ((error as NodeJS.ErrnoException).code === 'EINVAL') {
5555
this.#results[linkPath] = `File exists and is not a symbolic link`;
5656
} else {
5757
this.#results[linkPath] = `Can't link to ${ linkPath }: ${ error }`;
@@ -73,7 +73,7 @@ export default class UnixlikeIntegrations {
7373
const message = `Error creating symlink for ${ linkPath }:`;
7474

7575
console.error(message, err);
76-
if (isUnixFsError(err)) {
76+
if (err instanceof Error) {
7777
this.#results[linkPath] = `${ message } ${ err.message }`;
7878
}
7979

@@ -86,7 +86,7 @@ export default class UnixlikeIntegrations {
8686
const message = `Error unlinking symlink for ${ linkPath }`;
8787

8888
console.error(message, err);
89-
if (isUnixFsError(err)) {
89+
if (err instanceof Error) {
9090
this.#results[linkPath] = `${ message } ${ err.message }`;
9191
}
9292

@@ -113,7 +113,8 @@ export default class UnixlikeIntegrations {
113113
});
114114
}
115115
} catch (error) {
116-
if (!(isUnixFsError(error))) {
116+
// if (error typeof NodeJS.ErrnoException) {
117+
if (!(isNodeError(error))) {
117118
return;
118119
}
119120
switch (error.code) {

‎src/main/pathConflictManager.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import * as window from '@/window';
66
import pathConflict from '@/utils/pathConflict';
77
import Logging from '@/utils/logging';
88
import paths from '@/utils/paths';
9-
import { isUnixFsError } from '@/typings/unix.interface';
109

1110
const console = Logging.background;
1211
const DebounceInterval = 500; // msec
@@ -69,7 +68,7 @@ export default class PathConflictManager {
6968
try {
7069
await fs.promises.access(dirName, fs.constants.R_OK);
7170
} catch (err) {
72-
if (isUnixFsError(err) && err.code !== 'ENOENT') {
71+
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
7372
console.log(`error in setupPathWatchersForShadowing:`, err);
7473
}
7574
continue;

‎src/main/paths.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import Electron from 'electron';
1212

1313
import Logging from '@/utils/logging';
1414
import paths, { Paths } from '@/utils/paths';
15-
import { isUnixFsError } from '@/typings/unix.interface';
15+
import { isNodeError } from '@/typings/unix.interface';
1616

1717
const console = Logging.background;
1818
const APP_NAME = 'rancher-desktop';
@@ -114,7 +114,7 @@ function removeEmptyParents(directory: string) {
114114
try {
115115
fs.rmdirSync(parent);
116116
} catch (ex) {
117-
if (isUnixFsError(ex) && expectedErrors.includes(ex.code)) {
117+
if (isNodeError(ex) && expectedErrors.includes(ex.code)) {
118118
break;
119119
}
120120
throw ex;
@@ -132,7 +132,7 @@ function recursiveRemoveSync(target: string) {
132132
try {
133133
fs.rmSync(target, { recursive: true });
134134
} catch (ex) {
135-
if (isUnixFsError(ex) && ex.code === 'ENOENT') {
135+
if ((ex as NodeJS.ErrnoException).code === 'ENOENT') {
136136
return;
137137
}
138138
throw ex;
@@ -167,7 +167,7 @@ function tryRename(oldPath: string, newPath: string, info: string, deleteOnFailu
167167

168168
return 'succeeded';
169169
} catch (ex) {
170-
if (isUnixFsError(ex) && ['ENOENT', 'EEXIST'].includes(ex.code)) {
170+
if (isNodeError(ex) && ['ENOENT', 'EEXIST'].includes(ex.code)) {
171171
console.error(`Expected error moving ${ info }: ${ ex }`);
172172

173173
return 'failed';

‎src/main/update/LonghornProvider.ts‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import semver from 'semver';
1212

1313
import Logging from '@/utils/logging';
1414
import paths from '@/utils/paths';
15-
import { isUnixFsError } from '@/typings/unix.interface';
1615

1716
const console = Logging.update;
1817
const gCachePath = path.join(paths.cache, 'updater-longhorn.json');
@@ -147,7 +146,7 @@ export async function hasQueuedUpdate(): Promise<boolean> {
147146

148147
return true;
149148
} catch (error) {
150-
if (isUnixFsError(error) && error.code !== 'ENOENT') {
149+
if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
151150
console.error('Could not check for queued update:', error);
152151
}
153152
}
@@ -164,7 +163,7 @@ export async function setHasQueuedUpdate(isQueued: boolean): Promise<void> {
164163
await fs.promises.writeFile(gCachePath, JSON.stringify(cache),
165164
{ encoding: 'utf-8', mode: 0o600 });
166165
} catch (error) {
167-
if (isUnixFsError(error) && error.code !== 'ENOENT') {
166+
if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
168167
console.error('Could not check for queued update:', error);
169168
}
170169
}
@@ -218,7 +217,7 @@ export default class LonghornProvider extends Provider<UpdateInfo> {
218217
return cache;
219218
}
220219
} catch (error) {
221-
if (isUnixFsError(error) && error.code !== 'ENOENT') {
220+
if ((error as NodeJS.ErrnoException).code !== 'ENOENT') {
222221
// Log the unexpected error, but keep going.
223222
console.error('Error reading update cache:', error);
224223
}

‎src/typings/unix.interface.ts‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,15 @@ export const isUnixError = (val: any): val is UnixError => {
1212
'message' in val;
1313
};
1414

15-
export interface UnixFsError {
15+
export interface NodeError {
1616
syscall: string;
1717
path: string;
1818
code: string;
1919
message: string;
20+
errno: number;
2021
}
2122

22-
export const isUnixFsError = (val: any): val is UnixError => {
23+
export const isNodeError = (val: any): val is NodeError => {
2324
return 'syscall' in val &&
2425
'path' in val &&
2526
'code' in val &&

0 commit comments

Comments
 (0)