screwdriver-api 7.0.218 → 7.0.219

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "screwdriver-api",
-  "version": "7.0.218",
+  "version": "7.0.219",
   "description": "API server for the Screwdriver.cd service",
   "main": "index.js",
   "scripts": {

package/plugins/builds/README.md

@@ -75,12 +75,11 @@ Example payload:
 Arguments:
 
 * `type` - Return type for build artifact, `download` or `preview`
+* `dir` - If downloading directory or not (`true` or `false`, default `false`). Must be set with `type=download`.
 
 `GET /builds/{id}/artifacts/{name*}?type=preview`
 
-`GET /builds/{id}/artifacts/this/is/a/directory/path/?type=download`
-
-*Note: To download a directory, there must be a trailing slash (`/`) in the name and `type=download`.*
+`GET /builds/{id}/artifacts/this/is/a/directory/path?type=download&dir=true`
 
 #### Get build statuses
 `GET /builds/statuses`

package/plugins/builds/artifacts/get.js

@@ -50,9 +50,9 @@ module.exports = config => ({
                 .then(async () => {
                     // Directory should fetch manifest and
                     // gather all files that belong to that directory
-                    if (artifact.endsWith('/') && req.query.type === 'download') {
+                    if (req.query.dir && req.query.type === 'download') {
                         // Create a zip name from the directory structure
-                        const zipName = artifact.split('/').slice(-2)[0];
+                        const zipName = artifact.split('/').slice(-1)[0];
 
                         try {
                             const token = jwt.sign({
@@ -69,7 +69,7 @@ module.exports = config => ({
                                 method: 'GET'
                             }).text();
                             const manifestArray = manifest.trim().split('\n');
-                            const directoryArray = manifestArray.filter(f => f.startsWith(`./${artifact}`));
+                            const directoryArray = manifestArray.filter(f => f.startsWith(`./${artifact}/`));
 
                             // Create a stream and set up archiver
                             const archive = archiver('zip', { zlib: { level: 9 } });
@@ -100,7 +100,7 @@ module.exports = config => ({
                                         archive.emit('error', err); // Emit error to stop the archive process
                                     });
 
-                                    const relativePath = file.replace(`./${artifact}`, `./${zipName}/`);
+                                    const relativePath = file.replace(`./${artifact}/`, `./${zipName}/`);
 
                                     // Append the file stream to the archive with the correct relative path
                                     archive.append(fileStream, { name: relativePath });
@@ -168,7 +168,8 @@ module.exports = config => ({
                 name: artifactSchema
             }),
             query: joi.object({
-                type: typeSchema
+                type: typeSchema,
+                dir: joi.boolean().truthy('true').falsy('false').default(false)
             }).options({ allowUnknown: true })
         }
     }