fix: service keys must be lowercase

[lyager]

Services are registered with fullname as lowercase, when moving waiting_services to active after probe, the lookup failed due to using case service name

[lyager]

Maybe the better solution would be to create get/set on the my_services hash?

[keepsimple1]

Thanks for your PR! LGTM!

[keepsimple1]

Maybe the better solution would be to create get/set on the my_services hash?

Not sure about get/set on my_services, as it's internal data structure. I'm wondering maybe we should use lower case for the keys in the waiting_services to be consistent:

mdns-sd/src/service_info.rs

Line 807 in 661408d

pub(crate) waiting_services: HashSet<String>,

(I will merge your current change first)

[lyager]

Not sure about get/set on my_services, as it's internal data structure. I'm wondering maybe we should use lower case for the keys in the waiting_services to be consistent:

Consistency is always good :) There's also UniCase, but it might be going overboard. Thanks for the quick feedback.