Could we remove tls.client.server_name in favor of server.address

[lmolkova]

tls.client.server_name is defined as

semantic-conventions/docs/attributes-registry/tls.md

Line 25 in 5971366

| `tls.client.server_name` | string | Also called an SNI, this tells the server which hostname to which the client is attempting to connect to. | `opentelemetry.io` |  |

This should be the same as server.address. See https://community.f5.com/kb/technicalarticles/ssl-profiles-part-7-server-name-indication/279829

So the proposal is to deprecate the tls.client.server_name attribute.

[joaopgrassi]

CC @svrnm

[svrnm]

When I added this originally I took most attributes from Elastic Common Schema, so I had no other reason than that for adding it.

+1 for deprecation