screwdriver-api 7.0.229 → 7.0.230

package/package.json

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

package/plugins/pipelines/README.md

@@ -110,12 +110,19 @@ Query Params:
 * `sortBy` - *Optional* Field to sort by
 * `type` - *Optional* Get pipeline or pr events (default `pipeline`)
 * `prNum` - *Optional* Return only PR events of specified PR number
-* `sha` - *Optional* Search `sha` and `configPipelineSha` for events
 * `groupEventId` - *Optional* Return only events with a specified groupEventId
 * `id` - *Optional* Fetch specific event ID; alternatively can use greater than(`gt:`) or less than(`lt:`) prefix
+* `sha` - *Optional* Search `sha` and `configPipelineSha` for events
+* `author` - *Optional* Search commit author `username` and `name` for events
+* `creator` - *Optional* Search creator `username` and `name` for events
+* `message` - *Optional* Search commit `message` for events
+
+_Caveats_: Only one of the search fields can be used at one time (sha, author, creator, or message).
 
 `GET /pipelines/{id}/events?page={pageNumber}&count={countNumber}&sort={sort}&type={type}&prNum={prNumber}&sha={sha}`
 
+`GET /pipelines/{id}/events?message={message}`
+
 `GET /pipelines/{id}/events?id=gt:{eventId}&count={countNumber}` (greater than eventId)
 
 `GET /pipelines/{id}/events?id=lt:{eventId}&count={countNumber}&sort=ascending` (less than eventId)

package/plugins/pipelines/listEvents.js

@@ -34,7 +34,7 @@ module.exports = () => ({
 
         handler: async (request, h) => {
             const factory = request.server.app.pipelineFactory;
-            const { page, count, sha, prNum, id, sort, sortBy, groupEventId } = request.query;
+            const { page, count, sha, prNum, id, sort, sortBy, groupEventId, message, author, creator } = request.query;
 
             return factory
                 .get(request.params.id)
@@ -62,13 +62,18 @@ module.exports = () => ({
                         config.params.prNum = prNum;
                     }
 
+                    // Do a search
+                    // See https://www.w3schools.com/sql/sql_like.asp for syntax
                     if (sha) {
-                        config.search = {
-                            field: ['sha', 'configPipelineSha'],
-                            // Do a search for sha
-                            // See https://www.w3schools.com/sql/sql_like.asp for syntax
-                            keyword: `${sha}%`
-                        };
+                        config.search = { field: ['sha', 'configPipelineSha'], keyword: `${sha}%` };
+                    } else if (message) {
+                        config.search = { field: ['commit'], keyword: `%"message":"${message}%` };
+                    } else if (author) {
+                        // searches name and username
+                        config.search = { field: ['commit'], keyword: `%name":"${author}%` };
+                    } else if (creator) {
+                        // searches name and username
+                        config.search = { field: ['creator'], keyword: `%name":"${creator}%` };
                     }
 
                     if (groupEventId) {
@@ -95,15 +100,24 @@ module.exports = () => ({
                 id: pipelineIdSchema
             }),
             query: schema.api.pagination.concat(
-                joi.object({
-                    type: typeSchema,
-                    prNum: prNumSchema,
-                    sha: shaSchema,
-                    id: queryIdSchema,
-                    groupEventId: pipelineIdSchema,
-                    search: joi.forbidden(), // we don't support search for Pipeline list events
-                    getCount: joi.forbidden() // we don't support getCount for Pipeline list events
-                })
+                joi
+                    .object({
+                        type: typeSchema,
+                        prNum: prNumSchema,
+                        sha: shaSchema,
+                        message: joi.string().label('Commit message').example('fix: Typo'),
+                        author: joi.string().label('Author Name').example('Dao Lam'),
+                        creator: joi.string().label('Creator Name').example('Dao Lam'),
+                        id: queryIdSchema,
+                        groupEventId: pipelineIdSchema,
+                        search: joi.forbidden(), // we don't support search for Pipeline list events
+                        getCount: joi.forbidden() // we don't support getCount for Pipeline list events
+                    })
+                    // https://joi.dev/api/?v=17.13.3#objectoxorpeers-options
+                    .oxor('sha', 'message', 'author', 'creator')
+                    .messages({
+                        'object.oxor': 'You can only specify one search parameter: sha, message, author, or creator.'
+                    })
             )
         }
     }