screwdriver-api 8.0.137 → 8.0.139

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (2) hide show
  1. package/package.json +1 -1
  2. package/plugins/pipelines/sync.js +2 -4
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "screwdriver-api",
3
- "version": "8.0.137",
3
+ "version": "8.0.139",
4
4
  "description": "API server for the Screwdriver.cd service",
5
5
  "main": "index.js",
6
6
  "scripts": {
package/plugins/pipelines/sync.js CHANGED
@@ -73,9 +73,9 @@ module.exports = () => ({
73
73
  pipeline.admins = newAdmins;
74
74
  pipeline.adminUserIds = newAdminUserIds;
75
75
 
76
- await pipeline.update();
77
-
78
76
  if (!scope.includes('admin')) {
77
+ await pipeline.update();
78
+
79
79
  throw boom.forbidden(
80
80
  `User ${user.getFullDisplayName()} does not have push permission for this repo`
81
81
  );
@@ -97,8 +97,6 @@ module.exports = () => ({
97
97
  // This is needed to make admins dirty and update db
98
98
  pipeline.admins = newAdmins;
99
99
  pipeline.adminUserIds = newAdminUserIds;
100
-
101
- await pipeline.update();
102
100
  }
103
101
 
104
102
  try {