Return statements after sendBadge()

[jack126guy]

In server.js, I've noticed that there are return statements after some calls to sendBadge() but not after others. In my patch for Github license support, I've found that leaving out return leads to problems, so I've added them after all of the calls in my code.

TL;DR: Is return necessary after every call to sendBadge()?

[espadrine]

What must not happen are two calls to sendBadge() for a single request, so as to avoid sending the response twice (which actually simply raises an exception, which we don't want).

What we also want to avoid are doing work (sending requests, etc.) which needn't be done. In most cases, if there isn't a return after sendBadge(), the rest of the function will get executed, which is pointless because the badge was already delivered.