Skip to content
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 access tokens using github apps #1015

Closed
wants to merge 3 commits into from
Closed

support access tokens using github apps #1015

wants to merge 3 commits into from

Conversation

etwillbefine
Copy link

relates and maybe closes #418

This implementation uses ghinstallation as suggested by go-github.

When gh-app-integration-id and gh-app-private-key-path are present the ghinstallation Transport will be used as wrapper for http calls. The event parser needs access to the (originally in go-oauth called TokenSource) to retrieve an access token for cloning the repository for example. Read more: https://developer.github.com/apps/building-github-apps/authenticating-with-github-apps/#http-based-git-access-by-an-installation

This is more of an example and proof of concept. Tested it so far a little bit locally. Feedback would be much appreciated!

Todo:

  • add tests (once POC is approved)

Some clarification on wording: The App Integration equals the App ID. This points to the Github App. The Installation ID points to the installation within the Github Account or Organization. This setup will only support already authorized installations and requires the Installation ID (once App is authorized) to be present.

@etwillbefine
Copy link
Author

Sorry I overlooked the related PR #971 .. too bad :) But it seems already to be more thought out. Closing this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support installation as a GitHub App
1 participant