Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

feat: support sign up with information provided by OIDC providers #130

Merged
merged 15 commits into from
Apr 6, 2022

Conversation

rot1024
Copy link
Member

@rot1024 rot1024 commented Mar 28, 2022

Support any OIDC provider

  • Automatically retrieve user information from userinfo endpoint when signing up
  • Supports all JWT sub formats
  • Refactor some code to make them mockable

@rot1024 rot1024 marked this pull request as draft March 28, 2022 11:15
@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #130 (0effecf) into main (51178b3) will increase coverage by 0.36%.
The diff coverage is 47.23%.

❗ Current head 0effecf differs from pull request most recent head a375a5d. Consider uploading reports for the commit a375a5d to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   36.25%   36.61%   +0.36%     
==========================================
  Files         326      330       +4     
  Lines       29069    29254     +185     
==========================================
+ Hits        10538    10711     +173     
+ Misses      17576    17546      -30     
- Partials      955      997      +42     
Impacted Files Coverage Δ
internal/adapter/gql/context.go 0.00% <ø> (ø)
internal/adapter/gql/resolver_mutation_user.go 0.00% <0.00%> (ø)
internal/app/auth_server.go 0.00% <0.00%> (ø)
internal/app/jwt.go 0.00% <0.00%> (ø)
internal/app/web.go 0.00% <0.00%> (ø)
internal/infrastructure/auth0/authenticator.go 44.02% <ø> (-3.93%) ⬇️
internal/infrastructure/mailer/mock.go 0.00% <0.00%> (ø)
internal/infrastructure/memory/auth_request.go 0.00% <0.00%> (ø)
internal/infrastructure/memory/team.go 0.00% <0.00%> (ø)
internal/infrastructure/memory/user.go 0.00% <0.00%> (ø)
... and 12 more

@rot1024 rot1024 marked this pull request as ready for review March 30, 2022 12:46
@rot1024 rot1024 enabled auto-merge (squash) April 6, 2022 05:30
@rot1024 rot1024 merged commit fef60e6 into main Apr 6, 2022
@rot1024 rot1024 deleted the feat/oidc branch April 6, 2022 05:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant