From b73e018bdb6c2807270375e7e362d5b387b82729 Mon Sep 17 00:00:00 2001 From: ProgrammerIn-wonderland <3838shah@gmail.com> Date: Wed, 30 Oct 2024 15:12:53 -0400 Subject: [PATCH] bad broken symlink suppot I'll fix later --- src/api/Filesystem.ts | 2 +- src/api/LocalFS.ts | 58 ++++++++++++++++++++++++++++++++++++++++--- 2 files changed, 55 insertions(+), 5 deletions(-) diff --git a/src/api/Filesystem.ts b/src/api/Filesystem.ts index 6fb3bbaf..4b345bee 100644 --- a/src/api/Filesystem.ts +++ b/src/api/Filesystem.ts @@ -1205,7 +1205,7 @@ class AnuraFilesystem implements AnuraFSOperations { rmdir: (path: string) => this.processPath(path).promises.rmdir(path), stat: (path: string) => this.processPath(path).promises.stat(path), symlink: (srcPath: string, dstPath: string, type?: string) => - this.processPath(srcPath).promises.symlink(srcPath, dstPath, type), + this.processPath(dstPath).promises.symlink(srcPath, dstPath, type), truncate: (path: string, len: number) => this.processPath(path).promises.truncate(path, len), unlink: (path: string) => this.processPath(path).promises.unlink(path), diff --git a/src/api/LocalFS.ts b/src/api/LocalFS.ts index ca4ea65b..3e1a9a17 100644 --- a/src/api/LocalFS.ts +++ b/src/api/LocalFS.ts @@ -70,6 +70,7 @@ class LocalFS extends AFSProvider { path: string, recurseCounter = 0, ): Promise<[FileSystemDirectoryHandle, string]> { + console.log("Fetching dir handle for " + path); if (recurseCounter > 20) { throw { name: "ELOOP", @@ -89,6 +90,7 @@ class LocalFS extends AFSProvider { let acc = this.dirHandle; let curr = ""; for await (const part of path.split("/")) { + if (part === "") continue; curr += "/" + part; if ((this.stats.get(curr)?.mode & 0o170000) === 0o120000) { // We ran into a path symlink, we're storing symlinks of all types as files who's content is the target. @@ -106,7 +108,12 @@ class LocalFS extends AFSProvider { ); } } - + console.log( + "Getting dir handle " + + part + + " on " + + curr.split("/").slice(0, -1).join("/"), + ); acc = await acc.getDirectoryHandle(part); } return [acc, curr]; @@ -116,6 +123,10 @@ class LocalFS extends AFSProvider { options?: FileSystemGetFileOptions, recurseCounter = 0, ): Promise<[FileSystemFileHandle, string]> { + if (!path.includes("/")) { + path = "/" + path; + } + const parentFolder = path.slice(0, path.lastIndexOf("/")); const [parentHandle, realPath] = await this.getChildDirHandle(parentFolder); @@ -660,9 +671,12 @@ class LocalFS extends AFSProvider { readlink: async (path: string) => { // Check if the path exists in stats path = this.relativizePath(path); - path = Filer.Path.normalize(path); + const fileName = path.slice(path.lastIndexOf("/") + 1); + const [parentHandle, realParent] = await this.getChildDirHandle( + path.slice(0, path.lastIndexOf("/")), + ); + const stats = this.stats.get(realParent + "/" + fileName); - const stats = this.stats.get(path); if (!stats) { throw { name: "ENOENT", @@ -672,13 +686,49 @@ class LocalFS extends AFSProvider { stack: "Error: No such file", } as Error; } + if (!((stats.mode & 0o170000) === 0o120000)) { + throw { + // I think this is the wrong error type + name: "EINVAL", + code: "EINVAL", + errno: -22, + message: `Is not a symbolic link: ${path}`, + stack: "Error: Is not a symbolic link", + } as Error; + } // Return the target path - return stats.target; + return await ( + await (await parentHandle.getFileHandle(fileName)).getFile() + ).text(); }, symlink: async (target: string, path: string) => { // await this.promises.stat(path); // Save stats and resolve the promise + path = this.relativizePath(path); + if (target.startsWith("/")) { + target = this.relativizePath(target); + } + + const fileName = path.slice(path.lastIndexOf("/") + 1); + const [parentHandle, realParent] = await this.getChildDirHandle( + path.slice(0, path.lastIndexOf("/")), + ); + const fileHandleWritable = await ( + await parentHandle.getFileHandle(fileName, { create: true }) + ).createWritable(); + fileHandleWritable.write(target); + fileHandleWritable.close(); + + const fullPath = realParent + "/" + fileName; + const fileStats = this.stats.get(fullPath) || {}; + if (fullPath) { + fileStats.mode = 41380; + fileStats.ctimeMs = Date.now(); + fileStats.mtimeMs = Date.now(); + this.stats.set(path, fileStats); + } + await this.promises.saveStats(); return; },