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

Override configs deleted when updating connector #1336

Closed
alozano3 opened this issue Jan 5, 2023 · 1 comment
Closed

Override configs deleted when updating connector #1336

alozano3 opened this issue Jan 5, 2023 · 1 comment
Labels
bug Something isn't working connect Kafka Connect ui Need an ui update

Comments

@alozano3
Copy link
Contributor

alozano3 commented Jan 5, 2023

I am using JDBC connector. I have properties that override the underlying kafka producer, for example producer.override.sasl.jaas.config .

When I update the configs using the form, these properties get deleted. The problem is that the form retrieves the specific connector properties using the /config/valdiate endpoint of Connect REST API and those properties are not included. When updating the connector, those properties are not sent and get deleted.

I have tried to include those configs in the form or even hidden so they get sent. But I am not fluent in React so I guess there is a better option.

@tchiotludo
Copy link
Owner

Strange I've the feeling this one was already fix in the past, but can't find it on the code.
Mostly I think it's an equivalent of the one but any properties that that is not on validate.
PR are welcome !

@tchiotludo tchiotludo added bug Something isn't working ui Need an ui update connect Kafka Connect labels Jan 6, 2023
tchiotludo pushed a commit that referenced this issue Apr 4, 2023
…tab (#1344)

close #1336 

Co-authored-by: alozano3 <alozano@caixabanktech.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working connect Kafka Connect ui Need an ui update
Projects
Status: Done
Development

No branches or pull requests

2 participants