Skip to content

fix client metadata to always return an empty string for api_key#7960

Merged
thrau merged 1 commit intomasterfrom
fix-client-metadata
Mar 24, 2023
Merged

fix client metadata to always return an empty string for api_key#7960
thrau merged 1 commit intomasterfrom
fix-client-metadata

Conversation

@thrau
Copy link
Member

@thrau thrau commented Mar 24, 2023

in #7693 we changed the behavior of read_client_metadata to return None as key if pro is not available. previously it was always an empty string. our data schema, unfortunately, as api key as a non-nullable field, so this leads to rows landing in quarantine.

@thrau thrau requested a review from ackdav March 24, 2023 16:47
Copy link
Member

@ackdav ackdav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great catch! lgtm 🚀

@thrau thrau merged commit 52b03dd into master Mar 24, 2023
@thrau thrau deleted the fix-client-metadata branch March 24, 2023 16:59
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.

2 participants