feat: token-based authentication #1097
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## main #1097 +/- ##
==========================================
- Coverage 81.00% 79.62% -1.38%
==========================================
Files 26 35 +9
Lines 1927 2425 +498
==========================================
+ Hits 1561 1931 +370
- Misses 287 400 +113
- Partials 79 94 +15
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
chore(rpc): remove unused empty proto import feat(rpc/flipt/auth): add METHOD_NONE to Method enum feat(rpc/flipt/auth): drop METHOD_ prefix from enum names
fix(sql/auth): use crypto/rand.Reader in place of math/rand.New fix(sql/auth): capture test parameters in loop-body variables chore(migrations): drop index before dropping authentications table fix(import): run down migrations on --drop fix(migrations/sqlite3): syntax error in uniqueness constraint on temp table in down test(sql/auth): fuzz hashClientToken chore(sql/auth): more fuzzing seeds fix(sql/auth): map driver constraint errors to internal error representation chore(sql/auth): remove underscore from Fuzz test name chore(storage/sql): remove dead code refactor(migrations): change authentications method from string to integer refactor(storage/sql): move common field utilities into sql package chore(storage/sql): use keyed field in struct literals
…dTokenService (#1102) feat(auth/method/token): initial gRPC server implementation test(server/auth/method/token): assert token creation via API chore(server/auth): synchronize server stop and fatal on error fix(storage): use flipt/errors package chore(server/auth): validate invalid error adapts appropriately feat(authentication): wire up grpc service and gateway feat(server/auth): define unary server interceptor feat(storage/auth): implement list authentications feat(storage/auth): list with method predicate feat(auth): configure initial token bootstrap process chore(auth): correct documentation typos fix(proto): change create token http method PUT to POST
chore: empty commit to kick ci fix(auth/sql): add auth package qualifier when calling ListWithMethod chore(auth/sql): remove duplicate imports chore(auth/sql): adjust import naming to be consistent fix(sq/auth): ensure order defaults to ascending fix(sql): truncate timestamps to microseconds when inserting into database fix(sql): use precision 6 for mysql timestamp columns
fix: ensure authentication store is always instantiated refactor(storage): rename query params Validate or Normalize chore(rpc/auth): prefix method enum values with METHOD_ chore(sql/testing): use wait.ForSQL during mysql testcontainer boot chore(storage): increase list limit from 10 to 25 feat(openapi): add Authorization header security requirements feat(openapi): add security requirements
GeorgeMac
force-pushed
the
authentication
branch
from
07313d5
to
1b626c4
Compare
GeorgeMac
changed the title
markphelps
reviewed
Nov 9, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Supports #779
This is a (relatively) long-standing branch and PR. It is now ready for final review.
Each of the constituent PRs were reviewed into this branch though.
Here is the overview:
Authentication: Bearer ...header.PRs