I'd spread this out a little, and for consistency, pass an error as the first argument.

```js
async.map(groupsAgg, getBadgeInfo, function (err, badgeInfos) {
  callback(null, badgeInfos);
});
```

Note that when you change the callback invocation to pass an error first, you will have to change the call in the controller to also expect a first error argument.