getCurrentRequest method should only be called when request is actually needed

Problem/Motivation

@mglaman pointed out the possible problem of these three patterns in the class constructor in his blog:
$this->request = $request_stack->getCurrentRequest();
$this->nodeStorage = $entity_type_manager->getStorage('node');
$this->config = $config_factory->get('mymodule.settings');

https://git.drupalcode.org/project/migrate_log_ui/-/blob/1.0.x/src/Contr...