Consistent error handling and layering on two review findings

FixMediaService
Shipped
August 26, 2026 at 1:02 AM UTC
Author
Kamo
Commit
8b4c60a

Review findings on Tasks 14 and 15: - SystemBugController held a SystemBugRepository field and queried it directly for the new-bug count, while every other read in that class goes through SystemBugService — the one file with two ways to reach the same data. Added a one-line pass-through, SystemBugService.countNew(), and pointed the controller at it instead; the controller no longer holds a repository field. - **************** was the only endpoint in a 15+-endpoint file with no try/catch, unlike its own sibling webinarUnreviewedCount. Wrapped it the same way, returning a bare 500 rather than a JSON error body since the endpoint's return type is ResponseEntity<Void>. Finding 1 (Task 9's unguarded findBySession lookup) was fixed in the preceding commit, 36434f4.

All changes

Like what you see shipping?

Every one of these updates lands in your workspace automatically. Start free and watch it grow week after week.

Start Free ForeverView Pricing