github
/
OpenID-Connect-Java-Spring-Server
mirror of https://github.com/mitreid-connect/OpenID-Connect-Java-Spring-Server
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

DWN-39926 : use put rather than addAttribute for updating whitelists

pull/1601/head
Harry Smith 2023-01-04 10:04:04 +00:00
parent 0bea906f1e
commit 32240a5c5c
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
openid-connect-server/src/main/java/org/mitre/openid/connect/web/WhitelistAPI.java
View File

@ -147,9 +147,9 @@ public class WhitelistAPI {
whitelist = gson.fromJson(json, WhitelistedSite.class); whitelist = gson.fromJson(json, WhitelistedSite.class);
validateWhitelistScopes(whitelist.getAllowedScopes()); validateWhitelistScopes(whitelist.getAllowedScopes());
} catch (WhitelistScopesException e) { } catch (WhitelistScopesException e) {
logger.error("addNewWhitelistedSite failed due to WhitelistScopesException. {}", e.getMessage()); logger.error("updateWhitelistedSite failed due to WhitelistScopesException. {}", e.getMessage());
m.addAttribute(HttpCodeView.CODE, HttpStatus.BAD_REQUEST); m.put(HttpCodeView.CODE, HttpStatus.BAD_REQUEST);
m.addAttribute(JsonErrorView.ERROR_MESSAGE, "Could not save new whitelisted site. The server encountered a whitelist scopes exception. Contact a system administrator for assistance."); m.put(JsonErrorView.ERROR_MESSAGE, "Could not update whitelisted site. The server encountered a whitelist scopes exception. Contact a system administrator for assistance.");
return JsonErrorView.VIEWNAME; return JsonErrorView.VIEWNAME;
} catch (JsonParseException e) { } catch (JsonParseException e) {
logger.error("updateWhitelistedSite failed due to JsonParseException", e); logger.error("updateWhitelistedSite failed due to JsonParseException", e);