Support deserializing binary protobuf encoded message keys

[jdechicchis]

First off, Kafka UI is great, thanks for maintaining it!

I noticed this TODO in ProtobufFileRecordSerDe:

//TODO: currently we assume that keys for this serde are always string - need to discuss if it is ok

I think it would be nice to provide the ability to deserialize binary protobuf encoded message keys in addition to message values. To support this, I think a new configuration value such as Map<String, String> protobufMessageNameByTopicForKey could be added (open to suggestions here). In terms of how a message key is deserialized when using ProtobufFileRecordSerDe I feel this precedence order would make sense:

• If the protobuf message name is found in protobufMessageNameByTopicForKey deserialize the message key using a descriptor.
• Else, deserialize the message key as a string.

Happy to make a PR for this (I've been playing around with the code and doesn't seem like it would be terribly difficult to add), but wanted to discuss if adding such a configuration makes sense

[github-actions]

Hello there jdechicchis! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀

[Haarolean]

Hey, thanks for reaching out! Glad you like it.

Let's discuss and we'll see about the PR.

@iliax, what do you think?

[iliax]

I am ok with this. Will be happy to review PR, @jdechicchis