Skip to content

[Bug]: Remove Log::info in InitializedHandler.php to Prevent Production Storage Issues #50

@wikichua

Description

@wikichua

What happened?

Hi team,

I noticed that in the InitializedHandler class, the Log::info statement (lines 28-31) is logging every time a client is initialized. While this can be helpful for debugging, it could quickly lead to excessive log entries in a production environment, especially if client initializations happen frequently. This might cause storage and performance issues on the production server due to log bloat.

File: laravel-mcp-server/src/Server/Notification/InitializedHandler.php

Relevant Lines:
Start: Line 28
End: Line 31

Log::info('MCP Client Initialized', [
    'params' => $params,
    'initialized_at' => now()->toISOString(),
]);

How to reproduce the bug

Every when MCP server initialized it will happen.

Package Version

latest

PHP Version

8.3

Laravel Version

12

Which operating systems does this happen with?

No response

Notes

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions