Refactor the event subscriber so it returns the image directly without redirecting

Problem/Motivation

Currently, the module reacts to non-existing images by redirecting to the generation route. this causes an unnecessary redirect on the first request.

Proposed resolution

Refactor the code so the image is returned directly from the the subscriber, ideally moving helper functionality from the controller to a service class to keep the code cleaner.