Expose formatted service name as a const fn

[mkatychev]

Motivation

It is oftentimes cumbersome to use tonic::server::NamedService::NAME when T has multiple generics:

impl<T: ServerTrait> tonic::server::NamedService for ServiceServer<T> {
    const NAME: &'static str = "service_api.Service";
}

This is particularly egregious when using the HealthReporter methods since HealthReporter::set_serving requires a fully qualified ServiceServer<T>:

let (mut health_reporter, health_service) = tonic_health::server::health_reporter();
health_reporter.set_serving::<MyServer<MyInner<MyLongQualifiedType, MyOtherQualifiedType, T, T2>>>().await;

Solution

expose a pub const fn service_name() -> &'static str that returns the formatted string.

[AmeliasCode]

+1 to this! It would make it a lot less verbose to get the service name.

[djc]

I think it would still make sense for this to a const SERVICE_NAME: &str?

[mkatychev]

fair point, should the trait const below be const NAME: &'static str = SERVICE_NAME; then?

[djc]

Might be nice, want to make that change?

[mkatychev]

I cannot seem to reproduce this diff locally with protobuf 25:
https://github.com/hyperium/tonic/actions/runs/9472891207/job/26134316596#step:7:16

[LucioFranco]

@mkatychev you just need to run cargo run --package codegen

[mkatychev]

@mkatychev you just need to run cargo run --package codegen

Hi @LucioFranco I did that with the given version of protoc (25.1) on macOS and I am not able to produce the comment diff in the run log

[djc]

@mkatychev you just need to run cargo run --package codegen

Hi @LucioFranco I did that with the given version of protoc (25.1) on macOS and I am not able to produce the comment diff in the run log

Run cargo update locally before you run the codegen?

[djc]

Thanks!