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

Fix broken topic browsing when using config files without a 'schema-registry' section #576

Merged
merged 1 commit into from
Jan 31, 2021

Conversation

ebowden-tibco
Copy link
Contributor

As @xpr3sso discovered, my recent aada01e commit broke things if using an application config file that omitted the 'schema-registry' section entirely. This PR should fix that. (And also fixes an unrelated "toSchemasLastestVersion" -> "toSchemasLatestVersion" typo elsewhere that I happened to see.)

@xpr3sso - Please let me know if this works for you! Thanks for catching the bug!

…fig files without a 'schema-registry' section present were used.
@ebowden-tibco
Copy link
Contributor Author

Hm - bunch of unit tests failed in the automatic build that passed for me locally. @tchiotludo have any of the tests been flaky lately or should I look into this more?

@tchiotludo
Copy link
Owner

Test are flakky on the connect part. I restart it 😁

@xpr3sso
Copy link
Contributor

xpr3sso commented Jan 30, 2021

As @xpr3sso discovered, my recent aada01e commit broke things if using an application config file that omitted the 'schema-registry' section entirely. This PR should fix that. (And also fixes an unrelated "toSchemasLastestVersion" -> "toSchemasLatestVersion" typo elsewhere that I happened to see.)

@xpr3sso - Please let me know if this works for you! Thanks for catching the bug!

Works perfectly 👍 Thanks for having a look at this!

@tchiotludo tchiotludo merged commit fbc15eb into tchiotludo:dev Jan 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants