chore: use rand.Text and slog.DiscardHandler over intrernal implementation

[IAmSurajBobade]

Address TODOs in code requiring go 1.24 as minimum version

mcp/logging.go: use slog.DiscardHandler, over local implementation
mcp/util.go: use rand.Text, over local implementation

Fixes #772

[maciej-kisiel]

Thanks for the contribution, I have one comment.

[maciej-kisiel]

Could we replace calls to this function with rand.Text() and remove it altogether?

[IAmSurajBobade]

Thanks for the review, @maciej-kisiel. Addressed your review comment.

I did not remove randText method earlier, to minimize code change and also as ServerOptions.GetSessionID expects a func, and so anyway need to create anon func. And mcp/streamable.go, uses math/rand/v2, causing a conflict.

[IAmSurajBobade]

I did not remove randText method earlier, to minimize code change and ServerOptions.GetSessionID expects a func, and so anyway need to create anon func. Also mcp/streamable.go, uses math/rand/v2, causing a conflict and impacting readability.

[maciej-kisiel]

Just assigning rand.Text won't work?

[IAmSurajBobade]

No, it won't. SessionId expects func()string. I mentioned this in the response to your earlier review comment on why I reused the existing randText method earlier.

[maciej-kisiel]

rand.Text is func() string. Please do:

opts.GetSessionID = rand.Text

[IAmSurajBobade]

ahaa, my bad. resolved.

[maciej-kisiel]

Thanks for the contribution!