-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add GitHub OAuth config #511
Add GitHub OAuth config #511
Conversation
const githubAuthConfig: { [x: string]: any; } = { | ||
github: { | ||
githubWebUri: 'https://github.com', | ||
githubApiUri: 'https://api.github.com', | ||
clientID: 'clientID', | ||
clientSecret: 'clientSecret', | ||
oauthScopes: 'read:org,user:email', | ||
}, | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The huge diff is due to file name change. Actual diff only applies to these lines and last 5 files of this file.
Signed-off-by: Sayali Gaikawad <[email protected]>
0c0e133
to
eb12990
Compare
README.md
Outdated
1. Run with parameter with one of the following (refer [this](#ssl-configuration) for value of `useSsL`) - | ||
1. `npm run cdk deploy OpenSearch-CI-Dev -- -c runWithOidc=true -c useSsl=true` or, | ||
1. `cdk deploy OpenSearch-CI-Dev -c runWithOidc=true -c useSsl=true` | ||
1. `npm run cdk deploy OpenSearch-CI-Dev -- -c authType=oidc/github/default -c useSsl=true` or, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: oidc|github|default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jenkins github is OIDC plugin right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They define it as GitHub Auth plugin and it also has a config different that oidc in jenkins.yaml (configuration as code)
Signed-off-by: Sayali Gaikawad <[email protected]>
Signed-off-by: Sayali Gaikawad <[email protected]>
Description
[Breaking Change]
This PR replaces
runWithOidc
parameter withauthType
to support multiple authTypes.Defaults to whatever is written in
baseJenkins.yaml
.With this change we now support github as auth mechanism.
Issues Resolved
related #506. Will close this issue once we officially adapt this parameter in internal deployments as well.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.