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

Improve CachedAuthorizer #90

Merged
merged 1 commit into from
Aug 31, 2021
Merged

Improve CachedAuthorizer #90

merged 1 commit into from
Aug 31, 2021

Conversation

manicminer
Copy link
Owner

  • Export the Source authorizer to consumers can assert it
  • Ensure the Source is a struct pointer for consistency
  • Breaking: existing exported func CachedAuthorizer is renamed to NewCachedAuthorizer

@manicminer
Improve CachedAuthorizer
27ae44b 
- Export the CachedAuthorizer type and its Source authorizer to consumers can assert it
- Ensure the Source is a struct pointer for consistency
@manicminer manicminer added enhancement New feature or request breaking change Indicates a non-backwards compatible change package/auth labels Aug 31, 2021
@manicminer manicminer added this to the v0.26.0 milestone Aug 31, 2021
@manicminer manicminer merged commit 6cf21c0 into main Aug 31, 2021
@manicminer manicminer deleted the cached-authorizer-cleanup branch August 31, 2021 11:10
manicminer added a commit that referenced this pull request Aug 31, 2021
@manicminer
Changelog for #90
4d4ca96
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
breaking change Indicates a non-backwards compatible change enhancement New feature or request package/auth
Projects
None yet
Milestone
v0.26.0
Development

Successfully merging this pull request may close these issues.
1 participant
@manicminer