[email protected] (#11)

- fix .upmconfig.toml file access
RageAgainstThePixel · Aug 15, 2024 · 7df97b9 · 7df97b9
1 parent 0474800
commit 7df97b9
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 10 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -26238,9 +26238,14 @@ async function save_upm_config(registry_url, auth_token) {
     core.debug(`upm_config_toml_path: "${upm_config_toml_path}"`);
     const overwrite = core.getInput('overwrite') === 'true';
     try {
-        await fs.promises.access(upm_config_toml_path);
-        if (overwrite) {
-            await fs.promises.writeFile(upm_config_toml_path, '');
+        const fileHandle = await fs.promises.open(upm_config_toml_path, 'r');
+        try {
+            if (overwrite) {
+                await fs.promises.writeFile(upm_config_toml_path, '');
+            }
+        }
+        finally {
+            fileHandle.close();
         }
     }
     catch (error) {

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "upm-config",
-  "version": "2.0.3",
+  "version": "2.0.4",
   "description": "A GitHub Action for setting Unity Engine UPM private scoped registry credentials in CI/CD workflows.",
   "author": "RageAgainstThePixel",
   "license": "MIT",

diff --git a/src/upm_config.ts b/src/upm_config.ts
@@ -87,9 +87,13 @@ async function save_upm_config(registry_url: string, auth_token: string): Promis
     core.debug(`upm_config_toml_path: "${upm_config_toml_path}"`);
     const overwrite = core.getInput('overwrite') === 'true';
     try {
-        await fs.promises.access(upm_config_toml_path);
-        if (overwrite) {
-            await fs.promises.writeFile(upm_config_toml_path, '');
+        const fileHandle = await fs.promises.open(upm_config_toml_path, 'r');
+        try {
+            if (overwrite) {
+                await fs.promises.writeFile(upm_config_toml_path, '');
+            }
+        } finally {
+            fileHandle.close();
         }
     } catch (error) {
         await fs.promises.writeFile(upm_config_toml_path, '');