Merge pull request #2 from stilobique/master

Update dependency

README.md

@@ -1,5 +1,5 @@
 # Nextcloud Artifact Upload Action
-Upload artifacts to nextcloud and outputs a shareable URL.
+Upload artifacts to nextcloud and output a shareable URL.
 
 ### How it looks:
 ![image](https://user-images.githubusercontent.com/23460729/120891750-7f247380-c60a-11eb-9998-3b3b7f61066f.png)
@@ -23,7 +23,7 @@ jobs:
         with:
           name: 'my-artifact'                                   # Name of the artifact
           path: 'bin/**/*.exe'                                  # Globbing supported
-          nextcloud-url: 'https://nextcloud.example.com'        # Format of test results
+          nextcloud-url: 'https://nextcloud.example.com'        # Nextcloud URL
           nextcloud-username: ${{ secrets.NEXTCLOUD_USERNAME }} # Username from repository secret
           nextcloud-password: ${{ secrets.NEXTCLOUD_PASSWORD }} # Password from repository secret
 ```

action.yml

@@ -30,5 +30,5 @@ inputs:
     required: false
     default: ${{ github.token }}
 runs:
-  using: 'node12'
-  main: 'dist/index.js'
+  using: 'node20'
+  main: 'dist/index.js'

dist/index.js

@@ -367,6 +367,7 @@ class NextcloudArtifact {
         const client = new NextcloudClient_1.NextcloudClient(this.inputs.Endpoint, this.name, files.rootDirectory, this.inputs.Username, this.inputs.Password);
         try {
             const shareableUrl = await client.uploadFiles(files.filesToUpload);
+            core.setOutput('SHAREABLE_URL', shareableUrl);
             core.info(`Nextcloud shareable URL: ${shareableUrl}`);
             const resp = await this.octokit.rest.checks.update({
                 check_run_id: createResp.data.id,
@@ -486,7 +487,8 @@ class NextcloudClient {
         this.headers = { Authorization: 'Basic ' + Buffer.from(`${this.username}:${this.password}`).toString('base64') };
         this.davClient = webdav.createClient(`${this.endpoint.href}remote.php/dav/files/${this.username}`, {
             username: this.username,
-            password: this.password
+            password: this.password,
+            maxBodyLength: 1024 ** 3
         });
     }
     async uploadFiles(files) {
@@ -494,10 +496,15 @@ class NextcloudClient {
         const spec = this.uploadSpec(files);
         core.info('Zipping files...');
         const zip = await this.zipFiles(spec);
-        core.info('Uploading to Nextcloud...');
-        const filePath = await this.upload(zip);
-        core.info(`Remote file path: ${filePath}`);
-        return await this.shareFile(filePath);
+        try {
+            core.info('Uploading to Nextcloud...');
+            const filePath = await this.upload(zip);
+            core.info(`Remote file path: ${filePath}`);
+            return await this.shareFile(filePath);
+        }
+        finally {
+            await fs.unlink(zip);
+        }
     }
     uploadSpec(files) {
         const specifications = [];

src/nextcloud/NextcloudArtifact.ts

@@ -83,6 +83,7 @@ export class NextcloudArtifact {
 
     try {
       const shareableUrl = await client.uploadFiles(files.filesToUpload)
+      core.setOutput('SHAREABLE_URL', shareableUrl)
       core.info(`Nextcloud shareable URL: ${shareableUrl}`)
       const resp = await this.octokit.rest.checks.update({
         check_run_id: createResp.data.id,

src/nextcloud/NextcloudClient.ts

@@ -42,11 +42,14 @@ export class NextcloudClient {
     const spec = this.uploadSpec(files)
     core.info('Zipping files...')
     const zip = await this.zipFiles(spec)
-
-    core.info('Uploading to Nextcloud...')
-    const filePath = await this.upload(zip)
-    core.info(`Remote file path: ${filePath}`)
-    return await this.shareFile(filePath)
+    try {
+      core.info('Uploading to Nextcloud...')
+      const filePath = await this.upload(zip)
+      core.info(`Remote file path: ${filePath}`)
+      return await this.shareFile(filePath)
+    } finally {
+      await fs.unlink(zip)
+    }
   }
 
   private uploadSpec(files: string[]): FileSpec[] {
@@ -126,7 +129,7 @@ export class NextcloudClient {
     const remoteFilePath = `${remoteFileDir}/${this.artifact}.zip`
     core.debug(`Transferring file... (${file})`)
 
-    await this.davClient.putFileContents(remoteFilePath, await fs.readFile(file));
+    await this.davClient.putFileContents(remoteFilePath, await fs.readFile(file))
 
     return remoteFilePath
   }