Skip to content

feat: add github device flow for authentication #8232

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

Merged
merged 17 commits into from
Jun 29, 2023
Merged

feat: add github device flow for authentication #8232

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
4e9c234
feat: add github device flow for authentication
kylecarbs Jun 27, 2023
7654191
Improve askpass view
kylecarbs Jun 27, 2023
16ce6e1
Add routes to improve clarity of git auth
kylecarbs Jun 27, 2023
0af6a37
Redesign the git auth page
kylecarbs Jun 27, 2023
ae60ebf
Refactor to add a page view
kylecarbs Jun 28, 2023
8933b49
Fix sideways layout
kylecarbs Jun 28, 2023
18022ba
Remove legacy notify
kylecarbs Jun 28, 2023
fb5264c
Fix git auth redirects
kylecarbs Jun 28, 2023
2146bd2
Add E2E tests
kylecarbs Jun 28, 2023
d7cecd3
Fix route documentation
kylecarbs Jun 28, 2023
189d7fc
Fix imports
kylecarbs Jun 28, 2023
de5632c
Remove unused imports
kylecarbs Jun 28, 2023
847b088
Fix E2E web test
kylecarbs Jun 28, 2023
bdbb1db
Fix friendly message appearance
kylecarbs Jun 28, 2023
a4fcf4b
Fix layout shifting for full-screen sign-in
kylecarbs Jun 28, 2023
1ef7c54
Fix height going to 100%
kylecarbs Jun 28, 2023
d43bd71
Fix comments
kylecarbs Jun 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix imports
  • Loading branch information
@kylecarbs
kylecarbs committed Jun 28, 2023
commit 189d7fc2a0e0d2be965e9fb50524e204beb58a1c
7 changes: 0 additions & 7 deletions coderd/gitauth/config_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,13 +143,6 @@ func TestConvertYAML(t *testing.T) {
Type: string(codersdk.GitProviderGitHub),
}},
Error: "client_id must be provided",
}, {
Name: "NoClientSecret",
Input: []codersdk.GitAuthConfig{{
Type: string(codersdk.GitProviderGitHub),
ClientID: "example",
}},
Error: "client_secret must be provided",
}, {
Name: "DuplicateType",
Input: []codersdk.GitAuthConfig{{
Expand Down
16 changes: 10 additions & 6 deletions coderd/gitauth_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,12 +96,16 @@ func TestGitAuthByID(t *testing.T) {
AvatarURL: github.String("https://avatars.githubusercontent.com/u/12345678?v=4"),
})
case "/installs":
httpapi.Write(r.Context(), w, http.StatusOK, []github.Installation{{
ID: github.Int64(12345678),
Account: &github.User{
Login: github.String("coder"),
},
}})
httpapi.Write(r.Context(), w, http.StatusOK, struct {
Installations []github.Installation `json:"installations"`
}{
Installations: []github.Installation{{
ID: github.Int64(12345678),
Account: &github.User{
Login: github.String("coder"),
},
}},
})
}
}))
defer srv.Close()
Expand Down