Demote `Snapshot extension registration failed` to `DEBUG` instead of `WARN`

[Neurone]

I want to bring this conversation up again.

#24475 Changed the message from ERROR to WARN. I think this should be demoted to DEBUG.
I see many of these warnings in the log, and I ignore them - someone doesn't? - so they are not warnings anymore, just noise.

Because it's something related to other nodes, I cannot do anything about that. Is it correct?
Otherwise, how I should fix, or help to fix, those warnings?

[Neurone]

@ligi @nicjansma what do you think about this?

[karalabe]

This is caused - I think - by Optimism nodes who renamed the eth protocol in their network but not the snap. It's an issue that we're hitting these peers and we should fix it so the discovery does not give them back to us. Until then, it's unfortunately an issue in peering, even if harmless.

[Neurone]

This is caused - I think - by Optimism nodes who renamed the eth protocol in their network but not the snap. It's an issue that we're hitting these peers and we should fix it so the discovery does not give them back to us. Until then, it's unfortunately an issue in peering, even if harmless.

Ok, I see. But if that's just a reminder for developers to address an issue, then it's a bit too intrusive for the geth's common users to have it as a warning, even more, because it's actually harmless and they cannot do anything about that.

[kayagoban]

If it's harmless, why is it a Warning?