@zenfs/core 1.2.9 → 1.2.10

package/dist/emulation/sync.d.ts

@@ -64,7 +64,7 @@ export declare function writeFileSync(path: fs.PathOrFileDescriptor, data: FileC
  * Asynchronously append data to a file, creating the file if it not yet exists.
  * @option encoding Defaults to `'utf8'`.
  * @option mode Defaults to `0644`.
- * @option flag Defaults to `'a'`.
+ * @option flag Defaults to `'a+'`.
  */
 export declare function appendFileSync(filename: fs.PathOrFileDescriptor, data: FileContents, _options?: fs.WriteFileOptions): void;
 /**

package/dist/emulation/sync.js

@@ -272,12 +272,12 @@ writeFileSync;
  * Asynchronously append data to a file, creating the file if it not yet exists.
  * @option encoding Defaults to `'utf8'`.
  * @option mode Defaults to `0644`.
- * @option flag Defaults to `'a'`.
+ * @option flag Defaults to `'a+'`.
  */
 export function appendFileSync(filename, data, _options = {}) {
     const env_4 = { stack: [], error: void 0, hasError: false };
     try {
-        const options = normalizeOptions(_options, 'utf8', 'a', 0o644);
+        const options = normalizeOptions(_options, 'utf8', 'a+', 0o644);
         const flag = parseFlag(options.flag);
         if (!isAppendable(flag)) {
             throw new ErrnoError(Errno.EINVAL, 'Flag passed to appendFile must allow for appending.');

package/dist/file.js

@@ -1,5 +1,5 @@
-import { O_APPEND, O_CREAT, O_EXCL, O_RDONLY, O_RDWR, O_SYNC, O_TRUNC, O_WRONLY, S_IFMT, size_max } from './emulation/constants.js';
 import { config } from './emulation/config.js';
+import { O_APPEND, O_CREAT, O_EXCL, O_RDONLY, O_RDWR, O_SYNC, O_TRUNC, O_WRONLY, S_IFMT, size_max } from './emulation/constants.js';
 import { Errno, ErrnoError } from './error.js';
 import './polyfills.js';
 import { Stats } from './stats.js';

package/dist/inode.js

@@ -36,8 +36,8 @@ var __setFunctionName = (this && this.__setFunctionName) || function (f, name, p
     if (typeof name === "symbol") name = name.description ? "[".concat(name.description, "]") : "";
     return Object.defineProperty(f, "name", { configurable: true, value: prefix ? "".concat(prefix, " ", name) : name });
 };
+import { deserialize, serialize, sizeof, struct, types as t } from 'utilium';
 import { Stats } from './stats.js';
-import { types as t, struct, sizeof, serialize, deserialize } from 'utilium';
 /**
  * Room inode
  * @hidden
@@ -162,8 +162,8 @@ let Inode = (() => {
                 this.uid = stats.uid;
                 hasChanged = true;
             }
-            if (this.uid !== stats.uid) {
-                this.uid = stats.uid;
+            if (this.gid !== stats.gid) {
+                this.gid = stats.gid;
                 hasChanged = true;
             }
             if (this.atimeMs !== stats.atimeMs) {

package/dist/stats.js

@@ -1,5 +1,5 @@
 import { credentials } from './credentials.js';
-import { S_IFBLK, S_IFCHR, S_IFDIR, S_IFIFO, S_IFLNK, S_IFMT, S_IFREG, S_IFSOCK, S_IRWXG, S_IRWXO, S_IRWXU, size_max } from './emulation/constants.js';
+import { R_OK, S_IFBLK, S_IFCHR, S_IFDIR, S_IFIFO, S_IFLNK, S_IFMT, S_IFREG, S_IFSOCK, S_IRGRP, S_IROTH, S_IRUSR, S_IWGRP, S_IWOTH, S_IWUSR, S_IXGRP, S_IXOTH, S_IXUSR, size_max, W_OK, X_OK, } from './emulation/constants.js';
 /**
  * Provides information about a particular entry in the file system.
  * Common code used by both Stats and BigIntStats.
@@ -110,12 +110,37 @@ export class StatsCommon {
      */
     hasAccess(mode) {
         if (credentials.euid === 0 || credentials.egid === 0) {
-            //Running as root
+            // Running as root
             return true;
         }
-        // Mask for
-        const adjusted = (credentials.uid == this.uid ? S_IRWXU : 0) | (credentials.gid == this.gid ? S_IRWXG : 0) | S_IRWXO;
-        return (mode & this.mode & adjusted) == mode;
+        let perm = 0;
+        // Owner permissions
+        if (credentials.uid === this.uid) {
+            if (this.mode & S_IRUSR)
+                perm |= R_OK;
+            if (this.mode & S_IWUSR)
+                perm |= W_OK;
+            if (this.mode & S_IXUSR)
+                perm |= X_OK;
+        }
+        // Group permissions
+        if (credentials.gid === this.gid) {
+            if (this.mode & S_IRGRP)
+                perm |= R_OK;
+            if (this.mode & S_IWGRP)
+                perm |= W_OK;
+            if (this.mode & S_IXGRP)
+                perm |= X_OK;
+        }
+        // Others permissions
+        if (this.mode & S_IROTH)
+            perm |= R_OK;
+        if (this.mode & S_IWOTH)
+            perm |= W_OK;
+        if (this.mode & S_IXOTH)
+            perm |= X_OK;
+        // Perform the access check
+        return (perm & mode) === mode;
     }
     /**
      * Convert the current stats object into a credentials object

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@zenfs/core",
-	"version": "1.2.9",
+	"version": "1.2.10",
 	"description": "A filesystem, anywhere",
 	"funding": {
 		"type": "individual",

package/src/emulation/sync.ts

@@ -243,10 +243,10 @@ writeFileSync satisfies typeof fs.writeFileSync;
  * Asynchronously append data to a file, creating the file if it not yet exists.
  * @option encoding Defaults to `'utf8'`.
  * @option mode Defaults to `0644`.
- * @option flag Defaults to `'a'`.
+ * @option flag Defaults to `'a+'`.
  */
 export function appendFileSync(filename: fs.PathOrFileDescriptor, data: FileContents, _options: fs.WriteFileOptions = {}): void {
-	const options = normalizeOptions(_options, 'utf8', 'a', 0o644);
+	const options = normalizeOptions(_options, 'utf8', 'a+', 0o644);
 	const flag = parseFlag(options.flag);
 	if (!isAppendable(flag)) {
 		throw new ErrnoError(Errno.EINVAL, 'Flag passed to appendFile must allow for appending.');

package/src/file.ts

@@ -1,6 +1,6 @@
 import type { FileReadResult } from 'node:fs/promises';
-import { O_APPEND, O_CREAT, O_EXCL, O_RDONLY, O_RDWR, O_SYNC, O_TRUNC, O_WRONLY, S_IFMT, size_max } from './emulation/constants.js';
 import { config } from './emulation/config.js';
+import { O_APPEND, O_CREAT, O_EXCL, O_RDONLY, O_RDWR, O_SYNC, O_TRUNC, O_WRONLY, S_IFMT, size_max } from './emulation/constants.js';
 import { Errno, ErrnoError } from './error.js';
 import type { FileSystem } from './filesystem.js';
 import './polyfills.js';

package/src/inode.ts

@@ -1,5 +1,5 @@
+import { deserialize, serialize, sizeof, struct, types as t } from 'utilium';
 import { Stats, type StatsLike } from './stats.js';
-import { types as t, struct, sizeof, serialize, deserialize } from 'utilium';
 
 /**
  * Alias for an ino.
@@ -108,8 +108,8 @@ export class Inode implements StatsLike {
 			hasChanged = true;
 		}
 
-		if (this.uid !== stats.uid) {
-			this.uid = stats.uid;
+		if (this.gid !== stats.gid) {
+			this.gid = stats.gid;
 			hasChanged = true;
 		}
 

package/src/stats.ts

@@ -1,6 +1,28 @@
 import type * as Node from 'node:fs';
 import { credentials, type Credentials } from './credentials.js';
-import { S_IFBLK, S_IFCHR, S_IFDIR, S_IFIFO, S_IFLNK, S_IFMT, S_IFREG, S_IFSOCK, S_IRWXG, S_IRWXO, S_IRWXU, size_max } from './emulation/constants.js';
+import {
+	R_OK,
+	S_IFBLK,
+	S_IFCHR,
+	S_IFDIR,
+	S_IFIFO,
+	S_IFLNK,
+	S_IFMT,
+	S_IFREG,
+	S_IFSOCK,
+	S_IRGRP,
+	S_IROTH,
+	S_IRUSR,
+	S_IWGRP,
+	S_IWOTH,
+	S_IWUSR,
+	S_IXGRP,
+	S_IXOTH,
+	S_IXUSR,
+	size_max,
+	W_OK,
+	X_OK,
+} from './emulation/constants.js';
 
 /**
  * Indicates the type of a file. Applied to 'mode'.
@@ -222,13 +244,33 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 	 */
 	public hasAccess(mode: number): boolean {
 		if (credentials.euid === 0 || credentials.egid === 0) {
-			//Running as root
+			// Running as root
 			return true;
 		}
 
-		// Mask for
-		const adjusted = (credentials.uid == this.uid ? S_IRWXU : 0) | (credentials.gid == this.gid ? S_IRWXG : 0) | S_IRWXO;
-		return (mode & this.mode & adjusted) == mode;
+		let perm = 0;
+
+		// Owner permissions
+		if (credentials.uid === this.uid) {
+			if (this.mode & S_IRUSR) perm |= R_OK;
+			if (this.mode & S_IWUSR) perm |= W_OK;
+			if (this.mode & S_IXUSR) perm |= X_OK;
+		}
+
+		// Group permissions
+		if (credentials.gid === this.gid) {
+			if (this.mode & S_IRGRP) perm |= R_OK;
+			if (this.mode & S_IWGRP) perm |= W_OK;
+			if (this.mode & S_IXGRP) perm |= X_OK;
+		}
+
+		// Others permissions
+		if (this.mode & S_IROTH) perm |= R_OK;
+		if (this.mode & S_IWOTH) perm |= W_OK;
+		if (this.mode & S_IXOTH) perm |= X_OK;
+
+		// Perform the access check
+		return (perm & mode) === mode;
 	}
 
 	/**

package/tests/fs/stat.test.ts

@@ -1,5 +1,6 @@
 import assert from 'node:assert';
 import { suite, test } from 'node:test';
+import { credentials } from '../../dist/credentials.js';
 import { Stats } from '../../dist/stats.js';
 import { fs } from '../common.js';
 
@@ -34,6 +35,47 @@ suite('Stats', () => {
 		fs.close(fd);
 	});
 
+	test('hasAccess for non-root access', () => {
+		const newFile = 'new.txt';
+
+		fs.writeFileSync(newFile, 'hello', {
+			mode: 0o640, // allow group access
+		});
+
+		const prevCredentials = {
+			...credentials,
+		};
+		const uid = 33;
+		const nonRootCredentials = {
+			uid,
+			gid: uid,
+			euid: uid,
+			egid: uid,
+			suid: uid,
+			sgid: uid,
+		};
+
+		fs.chownSync(newFile, 0, nonRootCredentials.gid); // creating with root-user so that non-root user can access
+
+		Object.assign(credentials, nonRootCredentials);
+		const stat = fs.statSync(newFile);
+
+		assert.equal(stat.gid, nonRootCredentials.gid);
+		assert.equal(stat.uid, 0);
+		assert.equal(stat.hasAccess(fs.constants.R_OK), true);
+		assert.equal(stat.hasAccess(fs.constants.W_OK), false);
+		assert.equal(stat.hasAccess(fs.constants.X_OK), false);
+		// changing group
+
+		Object.assign(credentials, { ...nonRootCredentials, gid: 44 });
+
+		assert.equal(stat.hasAccess(fs.constants.R_OK), false);
+		assert.equal(stat.hasAccess(fs.constants.W_OK), false);
+		assert.equal(stat.hasAccess(fs.constants.X_OK), false);
+
+		Object.assign(credentials, prevCredentials);
+	});
+
 	test('stat file', async () => {
 		const stats = await fs.promises.stat(existing_file);
 		assert(!stats.isDirectory());

package/tests/fs/times.test.ts

@@ -5,14 +5,14 @@ import { ErrnoError } from '../../dist/error.js';
 import { _toUnixTimestamp } from '../../dist/utils.js';
 import { fs } from '../common.js';
 
-suite('times', () => {
-	const path = 'x.txt';
+const path = 'x.txt';
 
+suite('times', () => {
 	function expect_assert(resource: string | number, atime: Date | number, mtime: Date | number) {
 		const stats = typeof resource == 'string' ? fs.statSync(resource) : fs.fstatSync(resource);
 		// check up to single-second precision since sub-second precision is OS and fs dependent
-		assert(_toUnixTimestamp(atime) == _toUnixTimestamp(stats.atime));
-		assert(_toUnixTimestamp(mtime) == _toUnixTimestamp(stats.mtime));
+		assert.equal(_toUnixTimestamp(atime), _toUnixTimestamp(stats.atime));
+		assert.equal(_toUnixTimestamp(mtime), _toUnixTimestamp(stats.mtime));
 	}
 
 	async function runTest(atime: Date | number, mtime: Date | number): Promise<void> {
@@ -24,8 +24,7 @@ suite('times', () => {
 			assert.strictEqual(error.code, 'ENOENT');
 		});
 
-		// don't close this fd
-		const handle = await fs.promises.open(path, 'r');
+		await using handle = await fs.promises.open(path, 'r');
 
 		await handle.utimes(atime, mtime);
 		expect_assert(handle.fd, atime, mtime);
@@ -33,16 +32,6 @@ suite('times', () => {
 		fs.utimesSync(path, atime, mtime);
 		expect_assert(path, atime, mtime);
 
-		// some systems don't have futimes
-		// if there's an error, it be ENOSYS
-		try {
-			fs.futimesSync(handle.fd, atime, mtime);
-			expect_assert(handle.fd, atime, mtime);
-		} catch (error: any) {
-			assert(error instanceof ErrnoError);
-			assert.strictEqual(error.code, 'ENOSYS');
-		}
-
 		try {
 			fs.utimesSync('foobarbaz', atime, mtime);
 		} catch (error: any) {
@@ -59,11 +48,11 @@ suite('times', () => {
 	}
 
 	test('utimes works', async () => {
-		await runTest(new Date('1982/09/10 13:37:00'), new Date('1982/09/10 13:37:00'));
-		await runTest(new Date(), new Date());
-		await runTest(123456.789, 123456.789);
+		await test('new Date(...)', () => runTest(new Date('1982/09/10 13:37:00'), new Date('1982/09/10 13:37:00')));
+		await test('new Date()', () => runTest(new Date(), new Date()));
+		await test('number', () => runTest(123456.789, 123456.789));
 		const stats = fs.statSync(path);
-		await runTest(stats.atime, stats.mtime);
+		await test('from stats', () => runTest(stats.atime, stats.mtime));
 	});
 
 	test('read changes atime', async () => {