contrib-auto-doctrine: Warnings in a log

[samdark]

PHP Warning: DoctrineDBALDriverMiddlewareAbstractDriverMiddleware::connect(): OpenTelemetry: pre hook threw exception, class=DoctrineDBALDriverMiddlewareAbstractDriverMiddleware function=connect message=OpenTelemetryContribInstrumentationDoctrineAttributesResolver::getServerPort(): Return value must be of type string, int returned in Unknown on line 0

[samdark]

PHP 8.3, Doctrine 2.15.5

[Ardenexal]

I received the same warning. It was coming through in the response payload, so I've had to revert back to using the auto-pdo library for now. But looking forward to using this once the issue is fixed.

I think a simple solution should be updating the getServerPort to change the type to a string

before

    /**
     * Resolve attribute `server.port`
     */
    private static function getServerPort(array $params): string
    {
        return $params[1][0]['port'] ?? 'unknown';
    }

after

    /**
     * Resolve attribute `server.port`
     */
    private static function getServerPort(array $params): string
    {
        return (string)$params[1][0]['port'] ?? 'unknown';
    }

[DominicDetta]

I created the PR

[brettmc]

PR was merged, new version released.