Skip to content

Commit e77b106

Browse files
committed
Added TODO as placeholder for #144
1 parent 939d319 commit e77b106

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/main/java/aa/aggregators/institution/InstitutionAttributeAggregator.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,8 @@ public List<UserAttribute> aggregate(List<UserAttribute> input, Map<String, List
5858
String eduID = input.stream().filter(attribute -> attribute.getName().equals(EDU_ID))
5959
.findFirst().get().getValues().getFirst();
6060
String url = String.format("%s/api/attributes/%s", removeTrailingSlash(institutionEndpoint.getBaseURL()), eduID);
61+
//TODO: Use try / catch, and in case of error, default to the input user attributes.
62+
//See https://github.com/OpenConext/OpenConext-attribute-aggregation/issues/144
6163
Map<String, List<String>> body = restTemplate.getForEntity(url, Map.class).getBody();
6264

6365
LOG.debug("Received response {} from {} for SP {}", body, institutionEndpoint.getBaseURL(), spEntityID);

0 commit comments

Comments
 (0)