Skip to content

Add oauth providers via cli #4591

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 19 commits into from
Sep 12, 2018
Merged

Add oauth providers via cli #4591

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
3fb249b
add oauth providers via cli
techknowlogick Aug 1, 2018
c14da07
update text, and change command to `add`
techknowlogick Aug 23, 2018
8409b60
Merge branch 'master' into auth-cli
techknowlogick Aug 23, 2018
081c943
add update command, and list command
techknowlogick Aug 23, 2018
60613ed
add delete function
techknowlogick Aug 23, 2018
df3e058
allow updating of individual settings
techknowlogick Aug 23, 2018
0ea1516
add docs
techknowlogick Aug 23, 2018
f47d5f2
fix build errors
techknowlogick Aug 23, 2018
6de8c90
split oauth CLIs out of cmd to reduce duplication
techknowlogick Aug 24, 2018
23e5cac
Merge branch 'master' into auth-cli
techknowlogick Aug 24, 2018
c34ddea
Merge branch 'master' into auth-cli
techknowlogick Aug 28, 2018
91da63e
Merge branch 'master' into auth-cli
techknowlogick Aug 28, 2018
bf89682
Merge branch 'master' into auth-cli
techknowlogick Aug 30, 2018
4cc967e
Merge branch 'master' into auth-cli
techknowlogick Sep 4, 2018
20f65ba
Update admin.go
techknowlogick Sep 10, 2018
96c1062
Update admin.go
techknowlogick Sep 10, 2018
e3acadd
Merge branch 'master' into auth-cli
techknowlogick Sep 10, 2018
2aec1ab
Update admin.go
techknowlogick Sep 11, 2018
ba72d28
Merge branch 'master' into auth-cli
lafriks Sep 12, 2018
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
update text, and change command to add
  • Loading branch information
@techknowlogick
techknowlogick authored Aug 23, 2018
commit c14da072077a07807a7dca72463198b7643c3e6d
4 changes: 2 additions & 2 deletions cmd/admin.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ var (
}

microcmdAuthOauth = cli.Command{
Name: "oauth",
Name: "add-oauth",
Usage: "Add new Oauth authentication source",
Action: runAddOauth,
Flags: []cli.Flag{
Copy link
Member

Choose a reason for hiding this comment

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

Maybe this flag array could be moved out to array so that it could be reused in both add and update?

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks @lafriks updated.

Expand Down Expand Up @@ -180,7 +180,7 @@ var (
cli.StringFlag{
Name: "custom-auth-url",
Value: "",
Usage: "Use a custom Authorize URL (option for GitLab/GitHub)",
Usage: "Use a custom Authorization URL (option for GitLab/GitHub)",
},
cli.StringFlag{
Name: "custom-token-url",
Expand Down