Remove workflow from default OAuth scopes

mislav · mislav · commit 075b6f8aa684 · 2021-02-04T17:40:09.000+01:00
We now request it conditionally only for "HTTPS" login flow
diff --git a/internal/authflow/flow.go b/internal/authflow/flow.go
@@ -69,7 +69,7 @@ func authFlow(oauthHost string, IO *iostreams.IOStreams, notice string, addition
 		httpClient.Transport = api.VerboseLog(IO.ErrOut, logTraffic, IO.ColorEnabled())(httpClient.Transport)
 	}
 
-	minimumScopes := []string{"repo", "read:org", "gist", "workflow"}
+	minimumScopes := []string{"repo", "read:org", "gist"}
 	scopes := append(minimumScopes, additionalScopes...)
 
 	callbackURI := "http://127.0.0.1/callback"

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ func authFlow(oauthHost string, IO *iostreams.IOStreams, notice string, addition`
`69`	`69`	`httpClient.Transport = api.VerboseLog(IO.ErrOut, logTraffic, IO.ColorEnabled())(httpClient.Transport)`
`70`	`70`	`}`
`71`	`71`
`72`		`- minimumScopes := []string{"repo", "read:org", "gist", "workflow"}`
	`72`	`+ minimumScopes := []string{"repo", "read:org", "gist"}`
`73`	`73`	`scopes := append(minimumScopes, additionalScopes...)`
`74`	`74`
`75`	`75`	`callbackURI := "http://127.0.0.1/callback"`