summaryrefslogtreecommitdiffstats
path: root/LICENSE.txt
diff options
context:
space:
mode:
authorMichael Mokry <michael.mokry@att.com>2019-08-08 09:59:03 -0500
committerMichael Mokry <michael.mokry@att.com>2019-08-12 08:25:46 -0500
commit971903e3b96068094b35ffa4207f025b7b569e51 (patch)
treed2ebc569d8a00442c42fd4c3c3fe3bc940157023 /LICENSE.txt
parent70d02931707ebfa3c499ec1d9d1f7d67793b5d05 (diff)
Fix issue for policies not loading on GUI push tab
- found the problem to be with a line of code to add all the scopes from the role to a Set of scopes. when the scopes attribute in the role object is null it logs an exception and fails to load policies. I added a new method to check the value of the roles.getScopes() for null and only attempt to add the scope to the set if it is not null - I created a separate method to do this in order to avoid an increase in technical debt as it would have added to the complexity if I kept it in the same method. - PATCH 4: Added scopes to the method arguments in order to avoid overwriting scopes that are added in prevous interations of the loop. - PATCH 5: removed redundant scopes assignment per Jorge's review. Change-Id: Ifcc8775d3db8ecc722ee6806310a58ecb4e15856 Issue-ID: POLICY-1981 Signed-off-by: Michael Mokry <michael.mokry@att.com>
Diffstat (limited to 'LICENSE.txt')
0 files changed, 0 insertions, 0 deletions