Skip to content

Enforce user API capability limits at API key lookup time.#11453

Merged
vadimberezniker merged 5 commits intomasterfrom
consolidate_api_key_query_3
Mar 3, 2026
Merged

Enforce user API capability limits at API key lookup time.#11453
vadimberezniker merged 5 commits intomasterfrom
consolidate_api_key_query_3

Conversation

@vadimberezniker
Copy link
Member

Depends on #11452

@vadimberezniker vadimberezniker force-pushed the consolidate_api_key_query_3 branch from f7ee96c to ffd742c Compare March 2, 2026 22:39
@vadimberezniker vadimberezniker force-pushed the consolidate_api_key_query_2 branch from c6e13a9 to f077160 Compare March 2, 2026 22:39
Base automatically changed from consolidate_api_key_query_2 to master March 2, 2026 23:10
@vadimberezniker vadimberezniker force-pushed the consolidate_api_key_query_3 branch from ffd742c to b98523d Compare March 2, 2026 23:41
Copy link
Member Author

@vadimberezniker vadimberezniker left a comment

Choose a reason for hiding this comment

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

addressed feedback and added some doc comments to tables.APIKey

currently looking at the test failure

@vadimberezniker
Copy link
Member Author

test is fixed, i forgot to remove the code to delete user API key when user is removed from group.

@vadimberezniker vadimberezniker merged commit 5ac95ab into master Mar 3, 2026
13 checks passed
@vadimberezniker vadimberezniker deleted the consolidate_api_key_query_3 branch March 3, 2026 23:25
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