From 5466e68f502a15b8f0e698bc06ec4099ee4bf0d7 Mon Sep 17 00:00:00 2001 From: cong meng Date: Fri, 18 Jun 2021 11:30:18 +1200 Subject: [PATCH] fix(ACI): At least one team or user should be specified when creating a restricted container in Azure ACI EE-578 (#5204) Co-authored-by: Simon Meng --- .../create/createContainerInstanceController.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/app/azure/views/containerinstances/create/createContainerInstanceController.js b/app/azure/views/containerinstances/create/createContainerInstanceController.js index 7d73a4477..3ea1ef8ea 100644 --- a/app/azure/views/containerinstances/create/createContainerInstanceController.js +++ b/app/azure/views/containerinstances/create/createContainerInstanceController.js @@ -8,7 +8,8 @@ angular.module('portainer.azure').controller('AzureCreateContainerInstanceContro 'Notifications', 'Authentication', 'ResourceControlService', - function ($q, $scope, $state, AzureService, Notifications, Authentication, ResourceControlService) { + 'FormValidator', + function ($q, $scope, $state, AzureService, Notifications, Authentication, ResourceControlService, FormValidator) { var allResourceGroups = []; var allProviders = []; @@ -70,6 +71,11 @@ angular.module('portainer.azure').controller('AzureCreateContainerInstanceContro return 'At least one port binding is required'; } + const error = FormValidator.validateAccessControl(model.AccessControlData, Authentication.isAdmin()); + if (error !== '') { + return error; + } + return null; }