I'm looking for a way to get the hostname of a multisite, and I've found several answers about it.
One of the solutions was to use: Answer link
Per also a user comments that the use of the Request object can be dangerous, as mentioned in the following link.
The comment warning say: Url to documentation
Note: The use of this wrapper in particular is especially discouraged. Most code should not need to access the request directly. Doing so means it will only function when handling an HTTP request, and will require special modification or wrapping when run from a command line tool, from certain queue processors, or from automated tests.
If code must access the request, it is considerably better to register an object with the Service Container and give it a setRequest() method that is configured to run when the service is created. That way, the correct request object can always be provided by the container and the service can still be unit tested.
If this method must be used, never save the request object that is returned. Doing so may lead to inconsistencies as the request object is volatile and may change at various times, such as during a subrequest.
I have a custom module and I have a service, is it possible to implement it in a service?
My question is according to the comment and recommendation of the API, how is it that it should be implemented?