foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-12-22 15:24:46 -05:00
Code Issues Packages 1 Wiki Activity

Deno.chdir should require allow-read not allow-write (#5033)

Browse source
This commit is contained in:
Ryan Dahl 2020-05-02 18:33:43 -04:00 committed by GitHub
parent 2872b362ff
commit bbbf9f299c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 31 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
cli/js/lib.deno.ns.d.ts vendored
View file

@ -140,7 +140,7 @@ declare namespace Deno {
* Throws `Deno.errors.PermissionDenied` if the user does not have access * Throws `Deno.errors.PermissionDenied` if the user does not have access
* rights * rights
* *
* Requires --allow-write. * Requires --allow-read.
*/ */
export function chdir(directory: string): void; export function chdir(directory: string): void;

16
cli/js/tests/dir_test.ts
View file

@ -5,7 +5,9 @@ unitTest(function dirCwdNotNull(): void {
assert(Deno.cwd() != null); assert(Deno.cwd() != null);
}); });
unitTest({ perms: { write: true } }, function dirCwdChdirSuccess(): void { unitTest(
{ perms: { read: true, write: true } },
function dirCwdChdirSuccess(): void {
const initialdir = Deno.cwd(); const initialdir = Deno.cwd();
const path = Deno.makeTempDirSync(); const path = Deno.makeTempDirSync();
Deno.chdir(path); Deno.chdir(path);
@ -16,9 +18,10 @@ unitTest({ perms: { write: true } }, function dirCwdChdirSuccess(): void {
assertEquals(current, path); assertEquals(current, path);
} }
Deno.chdir(initialdir); Deno.chdir(initialdir);
}); }
);
unitTest({ perms: { write: true } }, function dirCwdError(): void { unitTest({ perms: { read: true, write: true } }, function dirCwdError(): void {
// excluding windows since it throws resource busy, while removeSync // excluding windows since it throws resource busy, while removeSync
if (["linux", "darwin"].includes(Deno.build.os)) { if (["linux", "darwin"].includes(Deno.build.os)) {
const initialdir = Deno.cwd(); const initialdir = Deno.cwd();
@ -39,7 +42,9 @@ unitTest({ perms: { write: true } }, function dirCwdError(): void {
} }
}); });
unitTest({ perms: { write: true } }, function dirChdirError(): void { unitTest(
{ perms: { read: true, write: true } },
function dirChdirError(): void {
const path = Deno.makeTempDirSync() + "test"; const path = Deno.makeTempDirSync() + "test";
try { try {
Deno.chdir(path); Deno.chdir(path);
@ -51,4 +56,5 @@ unitTest({ perms: { write: true } }, function dirChdirError(): void {
throw Error("raised different exception"); throw Error("raised different exception");
} }
} }
}); }
);

2
cli/ops/fs.rs
View file

@ -254,7 +254,7 @@ fn op_chdir(
) -> Result<JsonOp, OpError> { ) -> Result<JsonOp, OpError> {
let args: ChdirArgs = serde_json::from_value(args)?; let args: ChdirArgs = serde_json::from_value(args)?;
let d = PathBuf::from(&args.directory); let d = PathBuf::from(&args.directory);
state.check_write(&d)?; state.check_read(&d)?;
set_current_dir(&d)?; set_current_dir(&d)?;
Ok(JsonOp::Sync(json!({}))) Ok(JsonOp::Sync(json!({})))
} }