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.

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

feat(rpc/auth): add OIDC auth method service gRPC definition #1138

Merged
merged 13 commits into from
Dec 12, 2022

Conversation

GeorgeMac
Copy link
Contributor

@GeorgeMac GeorgeMac commented Nov 16, 2022

Supports #779

Fixes: FLI-46

This adds the gRPC definitions for the OIDC Authentication Method service.
Implementations will come in a later PR.

The two primary operations for this service are:

  1. AuthorizeURL

This operation will return a valid authorize URL for the requested provider.

  1. Callback

This operation will handle the callback from the authentication service redirect.
The result of this operation (if successful) will be a valid client token and authentication instance.

@codecov-commenter
Copy link

codecov-commenter commented Nov 16, 2022

Codecov Report

Merging #1138 (27dd224) into main (7bea8d9) will increase coverage by 0.21%.
The diff coverage is n/a.

❗ Current head 27dd224 differs from pull request most recent head 1725b07. Consider uploading reports for the commit 1725b07 to get more accurate results

@@            Coverage Diff             @@
##             main    #1138      +/-   ##
==========================================
+ Coverage   79.65%   79.87%   +0.21%     
==========================================
  Files          38       38              
  Lines        2758     2758              
==========================================
+ Hits         2197     2203       +6     
+ Misses        455      451       -4     
+ Partials      106      104       -2     
Impacted Files Coverage Δ
internal/storage/oplock/sql/sql.go 96.33% <0.00%> (+5.50%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@GeorgeMac GeorgeMac requested a review from a team as a code owner November 23, 2022 12:15
@@ -32,6 +35,106 @@
"application/json"
],
"paths": {
"/auth/v1/method/oidc/{provider}/authorize": {
"get": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

note: we should remove this generated swagger schema as well as the others in favor of the newer handwritten openapi3 schema used to power the docs in https:/flipt-io/flipt-openapi

FLI-107

@GeorgeMac GeorgeMac added the automerge Used by Kodiak bot to automerge PRs label Dec 12, 2022
@kodiakhq kodiakhq bot merged commit 0fa66db into main Dec 12, 2022
@kodiakhq kodiakhq bot deleted the gm/oidc-grpc-definition branch December 12, 2022 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Used by Kodiak bot to automerge PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants