-
Notifications
You must be signed in to change notification settings - Fork 62
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
Support Client ID as an alternative input to Client Secret #136
Comments
Just to clarify, you cannot use the Client Secret, the private key is still required. But the OAuth Client ID is now recommended to be used as a substitute to the numeric app ID for github.com, and probably GitHub Enterprise Server 3.14+ We just shipped support for passing the client ID to Octokit's app authentication strategy: |
@cloudshiftchris edit title to clarify that support is sought for using clientId instead of appId? @gr2m are you saying this is already supported via octokit/auth-app.js#606 ? |
yes |
So does this mean that we can pass in client ID instead of app ID, and this issue can be closed... or? @gr2m |
I think this should be closed. I have used clientId in place of appId |
GitHub now recommends using the clientId instead of the appId/privateKey
Augment the options to allow clientId/clientSecret, validating that either this pair or appId/privateKey are provided, and pass these options along in the call to
createAppAuth
.The text was updated successfully, but these errors were encountered: