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.

Sign up for GitHub

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

Jump to bottom

Kafka authentication protocol security.protocol field #963

Closed
snkas opened this issue Oct 31, 2023 · 1 comment · Fixed by #990
Closed

Kafka authentication protocol security.protocol field #963

snkas opened this issue Oct 31, 2023 · 1 comment · Fixed by #990
Assignees
@Karakatiza666
Labels
bug Something isn't working enhancement New feature or request Web Console Related to the browser based UI

Comments

@snkas
Copy link
Contributor

snkas commented Oct 31, 2023

The security.protocol field should have its members be capitalized (PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL) (documentation).
If the security.protocol is capitalized currently (through the Python API), the UI goes blank and displays "Application error: a client-side exception has occurred (see the browser console for more information)." when clicking the "Edit" button (browser console shows a "ValiError: Invalid type").
The connector works with both the capitalized and non-capitalized version.
@snkas snkas added bug Something isn't working enhancement New feature or request Web Console Related to the browser based UI labels Oct 31, 2023
@snkas
Copy link
Contributor Author

snkas commented Oct 31, 2023

(Low priority, as the current workaround is to only use the lowercase versions, which works)

@Karakatiza666 Karakatiza666 self-assigned this Nov 6, 2023
Karakatiza666 added a commit that referenced this issue Nov 8, 2023
@Karakatiza666
Capitalize Kafka authentication protocol security.protocol field: #963
b7fe812 
Signed-off-by: George <bulakh.96@gmail.com>
@Karakatiza666 Karakatiza666 mentioned this issue Nov 8, 2023
Merged
Karakatiza666 added a commit that referenced this issue Nov 13, 2023
@Karakatiza666
Capitalize Kafka authentication protocol security.protocol field: #963
d92b69b 
Signed-off-by: George <bulakh.96@gmail.com>
Karakatiza666 added a commit that referenced this issue Nov 14, 2023
@Karakatiza666
Capitalize Kafka authentication protocol security.protocol field: #963
4f04cdb 
Signed-off-by: George <bulakh.96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Karakatiza666 Karakatiza666

Labels
bug Something isn't working enhancement New feature or request Web Console Related to the browser based UI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Multiple commits with UI fixes, see description feldera/feldera
2 participants
@snkas @Karakatiza666