Skip to content

Bug fixes for the ApiKeyProvider#4

Merged
intentionally-left-nil merged 6 commits intomainfrom
bug-fixes
Aug 12, 2025
Merged

Bug fixes for the ApiKeyProvider#4
intentionally-left-nil merged 6 commits intomainfrom
bug-fixes

Conversation

@intentionally-left-nil
Copy link
Contributor

When actually implementing the ApiKeyProvider, several type issues were discovered. This PR fixes:

  • The api_key providers entrypoint is exported so it can be imported directly
  • RuntError options are optional, and the empty {} can be omitted
  • ListApiKeys takes in a limit/offset value
  • ApiKeyProvider passes back an AuthenticatedContext, to prevent unecessary non-null assertions
  • the index exports all the values from shared, not just the types

@intentionally-left-nil intentionally-left-nil merged commit c7aad31 into main Aug 12, 2025
2 checks passed
@intentionally-left-nil intentionally-left-nil deleted the bug-fixes branch August 12, 2025 00:09
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.

1 participant