Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Serde: HTTP 500 when proto files contain Any type #650

Open
4 tasks done
jarasez opened this issue Nov 7, 2024 · 3 comments · May be fixed by #696
Open
4 tasks done

Serde: HTTP 500 when proto files contain Any type #650

jarasez opened this issue Nov 7, 2024 · 3 comments · May be fixed by #696
Assignees
Labels
area/serde Serialization & Deserialization (plugins) scope/backend Related to backend changes status/triage/completed Automatic triage completed type/bug Something isn't working
Milestone

Comments

@jarasez
Copy link

jarasez commented Nov 7, 2024

Issue submitter TODO list

  • I've looked up my issue in FAQ
  • I've searched for an already existing issues here
  • I've tried running main-labeled docker image and the issue still persists there
  • I'm running a supported version of the application which is listed here

Describe the bug (actual behavior)

I have the following protofiles structure:
protofiles/common/main.proto
protofiles/ref/referenced.proto

main.proto:
`syntax = "proto3";
import "google/protobuf/any.proto";

package common;

message Main {
string name = 1;
google.protobuf.Any payload = 2;
}`

referenced.proto
`syntax = "proto3";
import "google/protobuf/any.proto";

package ref;

message Referenced {
string id = 1;
}`

The setup for the docker image for the serde is:
kafka.clusters.0.serde.0.name: ProtobufFile kafka.clusters.0.serde.0.topicValuesPattern: "test*" kafka.clusters.0.serde.0.PROPERTIES.protobufFilesDir: /protofiles/ kafka.clusters.0.serde.0.properties.protobufMessageName: common.Main

When I try the Produce Message, I get the error that Referenced type is not found.
Screenshot 2024-11-07 at 09 00 49

Expected behavior

What should be the correct way to use proto Any types?

Your installation details

I am using docker image: ghcr.io/kafbat/kafka-ui:latest

Steps to reproduce

proto files and configuration are above

Screenshots

No response

Logs

No response

Additional context

No response

@jarasez jarasez added status/triage Issues pending maintainers triage type/bug Something isn't working labels Nov 7, 2024
@kapybro kapybro bot added status/triage/manual Manual triage in progress area/serde Serialization & Deserialization (plugins) status/triage/completed Automatic triage completed and removed status/triage Issues pending maintainers triage labels Nov 7, 2024
Copy link

github-actions bot commented Nov 7, 2024

Hi jarasez! 👋

Welcome, and thank you for opening your first issue in the repo!

Please wait for triaging by our maintainers.

As development is carried out in our spare time, you can support us by sponsoring our activities or even funding the development of specific issues.
Sponsorship link

If you plan to raise a PR for this issue, please take a look at our contributing guide.

@DimaVilda
Copy link

hi @jarasez
I was able to reproduce a bug by writing an integration test and I found the issue. Will try to create a bugfix as soon as I have some free time for it

@DimaVilda
Copy link

can you assign me to this issue ?

@DimaVilda DimaVilda linked a pull request Dec 8, 2024 that will close this issue
13 tasks
@Haarolean Haarolean added scope/backend Related to backend changes and removed status/triage/manual Manual triage in progress labels Dec 9, 2024
@Haarolean Haarolean moved this to In Review in Up for grabs Dec 9, 2024
@Haarolean Haarolean added this to the 1.1 milestone Dec 18, 2024
@Haarolean Haarolean moved this from Todo to PR Approved in Release 1.1 Dec 18, 2024
@Haarolean Haarolean changed the title ProtobufFile serde with proto files containing Any Serde: HTTP 500 when proto files contain Any type Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/serde Serialization & Deserialization (plugins) scope/backend Related to backend changes status/triage/completed Automatic triage completed type/bug Something isn't working
Projects
Status: PR Approved
Status: In Review
Development

Successfully merging a pull request may close this issue.

3 participants