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

feat: Implement auth module and middleware #109

Merged
merged 8 commits into from
Oct 26, 2022
Merged

feat: Implement auth module and middleware #109

merged 8 commits into from
Oct 26, 2022

Conversation

v3g42
Copy link
Contributor

@v3g42 v3g42 commented Oct 19, 2022 •
edited

Implements the following

  1. Api Auth Middleware
  • Validates APIs
  • Adds access to CacheReader
  1. POST /auth/token
  • API to generate tokens for cache read only access
  1. Authorizer lib
  • Responsible for generating and validating JWT tokens
  1. CacheReader
  • Read only wrapper on top of Cache
  • Adds dynamic filters on top of get, query APIs.

@v3g42 v3g42 changed the title Feat/auth feat: Implement auth module and middleware Oct 25, 2022
@v3g42 v3g42 marked this pull request as ready for review October 25, 2022 19:42
@duonganhthu43 duonganhthu43 merged commit b150f07 into main Oct 26, 2022
@duonganhthu43 duonganhthu43 deleted the feat/auth branch October 26, 2022 03:33
@v3g42
Copy link
Contributor Author

v3g42 commented Oct 26, 2022

Fixes #93

@v3g42 v3g42 mentioned this pull request Oct 26, 2022
Closed
5 tasks
@v3g42 v3g42 linked an issue Nov 2, 2022 that may be closed by this pull request
Implement auth based on document keys #153
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@duonganhthu43 duonganhthu43 duonganhthu43 approved these changes

@snork-alt snork-alt Awaiting requested review from snork-alt

@karolisg karolisg Awaiting requested review from karolisg

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement auth based on document keys
2 participants
@v3g42 @duonganhthu43