libre.sh/keycloak-scim
3
0
Fork 0
Code Issues 15 Pull requests Releases 1 Packages Activity Actions

Merge pull request #417 from mposolda/master

Browse source 
At least one redirect_uri must be available for oauthClients and applica...
This commit is contained in:
Stian Thorgersen 2014-05-23 10:11:06 +01:00
commit 142b155460
2 changed files with 35 additions and 31 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
forms/common-themes/src/main/resources/theme/admin/base/resources/js/controllers/applications.js
View file

@ -263,10 +263,10 @@ module.controller('ApplicationDetailCtrl', function($scope, realm, application,
} }
$scope.save = function() { $scope.save = function() {
if ($scope.create) { if (!$scope.application.bearerOnly && (!$scope.application.redirectUris || $scope.application.redirectUris.length == 0)) {
if (!$scope.application.bearerOnly && (!$scope.application.redirectUris || $scope.application.redirectUris.length == 0)) { Notifications.error("You must specify at least one redirect uri");
Notifications.error("You must specify at least one redirect uri"); } else {
} else { if ($scope.create) {
Application.save({ Application.save({
realm: realm.realm, realm: realm.realm,
application: '' application: ''
@ -277,16 +277,16 @@ module.controller('ApplicationDetailCtrl', function($scope, realm, application,
$location.url("/realms/" + realm.realm + "/applications/" + id); $location.url("/realms/" + realm.realm + "/applications/" + id);
Notifications.success("The application has been created."); Notifications.success("The application has been created.");
}); });
} else {
Application.update({
realm : realm.realm,
application : application.name
}, $scope.application, function() {
$scope.changed = false;
application = angular.copy($scope.application);
Notifications.success("Your changes have been saved to the application.");
});
} }
} else {
Application.update({
realm : realm.realm,
application : application.name
}, $scope.application, function() {
$scope.changed = false;
application = angular.copy($scope.application);
Notifications.success("Your changes have been saved to the application.");
});
} }
}; };

40
forms/common-themes/src/main/resources/theme/admin/base/resources/js/controllers/oauth-clients.js
View file

@ -133,25 +133,29 @@ module.controller('OAuthClientDetailCtrl', function($scope, realm, oauth, OAuthC
} }
$scope.save = function() { $scope.save = function() {
if ($scope.create) { if (!$scope.oauth.bearerOnly && (!$scope.oauth.redirectUris || $scope.oauth.redirectUris.length == 0)) {
OAuthClient.save({ Notifications.error("You must specify at least one redirect uri");
realm: realm.realm
}, $scope.oauth, function (data, headers) {
$scope.changed = false;
var l = headers().location;
var id = l.substring(l.lastIndexOf("/") + 1);
$location.url("/realms/" + realm.realm + "/oauth-clients/" + id);
Notifications.success("The oauth client has been created.");
});
} else { } else {
OAuthClient.update({ if ($scope.create) {
realm : realm.realm, OAuthClient.save({
id : oauth.id realm: realm.realm
}, $scope.oauth, function() { }, $scope.oauth, function (data, headers) {
$scope.changed = false; $scope.changed = false;
oauth = angular.copy($scope.oauth); var l = headers().location;
Notifications.success("Your changes have been saved to the oauth client."); var id = l.substring(l.lastIndexOf("/") + 1);
}); $location.url("/realms/" + realm.realm + "/oauth-clients/" + id);
Notifications.success("The oauth client has been created.");
});
} else {
OAuthClient.update({
realm : realm.realm,
id : oauth.id
}, $scope.oauth, function() {
$scope.changed = false;
oauth = angular.copy($scope.oauth);
Notifications.success("Your changes have been saved to the oauth client.");
});
}
} }
}; };