refactor: Avoid NodeJS-specific module (use explicit logger scopes) (#3309)

This pull request standardizes logger initialization throughout the
codebase by replacing the use of `new Logger(module)` with a more
descriptive string identifier for each module or class. This improves
the clarity of log messages and makes it easier to trace logs back to
their source components.

### Changes

**Logger initialization updates (all changes):**

* Replaced `new Logger(module)` with `new Logger('<ClassOrFileName>')`
in all relevant files across `autocertifier-client`,
`autocertifier-server`, `cdn-location`, and `dht` packages to ensure
consistent and descriptive logging.

* Replaced all occurrences of `loggerFactory.createLogger(module)` with
`loggerFactory.createLogger('<ClassOrFileName>')` in all relevant files
across the monorepo.

### Future steps

> [!NOTE]
> Because logger instances are created at the top level of each module,
simply importing the package into a host project will **eagerly
instantiate all associated loggers**, even if the corresponding exports
are never used.

1. Many modules currently instantiate a logger at import time (e.g. `new
Logger('Foo')`). This introduces a side effect during module evaluation,
which prevents effective tree-shaking of both the module itself and the
`Logger` implementation.

A future improvement would be to decouple logger creation from module
imports by moving initialization behind explicit factory functions or
lazy accessors. This would reduce import-time side effects and allow
unused modules (and the logger code) to be safely tree-shaken by the
bundler.

2. `Logger` was the key reason why we kept using (and polyfilling)
`module`. Consumers will be able to drop that env-specific "dependency"
from their setups soon.

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mondoreale <320066+mondoreale@users.noreply.github.com>

Author: mondoreale

packages/autocertifier-client/src/AutoCertifierClient.ts

@@ -15,7 +15,7 @@ interface AutoCertifierClientEvents {
 
 export type HasSession = (request: HasSessionRequest, context: ServerCallContext) => Promise<HasSessionResponse>
 
-const logger = new Logger(module)
+const logger = new Logger('AutoCertifierClient')
 
 const ensureConfigFileWritable = (directory: string): void => {
     const baseDirectory = getBaseDirectory(directory)

packages/autocertifier-client/src/RestClient.ts

@@ -5,7 +5,7 @@ import { CreateCertifiedSubdomainRequest } from './data/CreateCertifiedSubdomain
 import { Logger } from '@streamr/utils'
 import { makeHttpRequest } from './makeHttpRequest'
 
-const logger = new Logger(module)
+const logger = new Logger('RestClient')
 
 export class RestClient {
 

packages/autocertifier-server/bin/copyARecordsToRoute53.ts

@@ -6,7 +6,7 @@ import chunk from 'lodash/chunk'
 import { ChangeAction } from '@aws-sdk/client-route-53'
 
 (async () => {
-    const logger = new Logger(module)
+    const logger = new Logger('copyARecordsToRoute53')
 
     const domainName = validateEnvironmentVariable('AUTOCERTIFIER_DOMAIN_NAME')
     const databaseFilePath = validateEnvironmentVariable('AUTOCERTIFIER_DATABASE_FILE_PATH')

packages/autocertifier-server/src/AutoCertifierServer.ts

@@ -12,7 +12,7 @@ import { ChallengeManager } from './ChallengeManager'
 import { RRType } from '@aws-sdk/client-route-53'
 import { Route53Api } from './Route53Api'
 
-const logger = new Logger(module)
+const logger = new Logger('AutoCertifierServer')
 
 export const validateEnvironmentVariable = (name: string): string | never => {
     const value = process.env[name]

packages/autocertifier-server/src/CertificateCreator.ts

@@ -5,7 +5,7 @@ import path from 'path'
 import { ChallengeManager } from './ChallengeManager'
 import { Challenge } from 'acme-client/types/rfc8555'
 
-const logger = new Logger(module)
+const logger = new Logger('CertificateCreator')
 
 // https://letsencrypt.org/docs/challenge-types/#dns-01-challenge
 const DNS_01_CHALLENGE = 'dns-01'

packages/autocertifier-server/src/Database.ts

@@ -5,7 +5,7 @@ import { open, Statement, Database as SqliteDatabase } from 'sqlite'
 import { Logger, filePathToNodeFormat } from '@streamr/utils'
 import { DatabaseError, InvalidSubdomainOrToken } from '@streamr/autocertifier-client'
 
-const logger = new Logger(module)
+const logger = new Logger('Database')
 
 export class Database {
 

packages/autocertifier-server/src/DnsServer.ts

@@ -8,7 +8,7 @@ type AsyncDnsHandler = (
     ...args: Parameters<DnsHandler>
 ) => Promise<void>
 
-const logger = new Logger(module)
+const logger = new Logger('DnsServer')
 
 // https://help.dnsfilter.com/hc/en-us/articles/4408415850003-DNS-Return-Codes
 // DNS Query Format Error

packages/autocertifier-server/src/RestServer.ts

@@ -14,7 +14,7 @@ import bodyParser from 'body-parser'
 import * as https from 'https'
 import * as fs from 'fs'
 
-const logger = new Logger(module)
+const logger = new Logger('RestServer')
 
 type ExpressType = ReturnType<typeof express>
 type ServerType = ReturnType<ExpressType['listen']>

packages/autocertifier-server/src/StreamrChallenger.ts

@@ -15,7 +15,7 @@ import { toProtoRpcClient } from '@streamr/proto-rpc'
 import { Logger } from '@streamr/utils'
 import { FailedToConnectToStreamrWebSocket, AutoCertifierRpcClient, SERVICE_ID } from '@streamr/autocertifier-client'
 
-const logger = new Logger(module)
+const logger = new Logger('StreamrChallenger')
 
 // This is a dummy peer descriptor that is used to connect to the streamr websocket
 // To ensure that the autocertified subdomain is used for the Streamr Network

packages/cdn-location/src/fetchAirportCodeFromCdn.ts

@@ -1,6 +1,6 @@
 import { Logger, withTimeout } from '@streamr/utils'
 
-const logger = new Logger(module)
+const logger = new Logger('fetchAirportCodeFromCdn')
 
 const fetchHeader: (url: string, header: string, timeout: number) => Promise<string | null> = async (
     url: string,