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 Personal Access Tokens with Github Enterprise #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

luka5
Copy link

@luka5 luka5 commented Jul 28, 2021

The RUNNER_REPOSITORY_URL and RUNNER_ORGANIZATION_URL combined with a RUNNER_TOKEN already support any Github Enterprise. Unfortunately, the recommended GITHUB_ACCESS_TOKEN does connect with api.github.com directly. Therefore one can not use this project together with a Github Enterprise instance.

With this PR, the Github API is automatically derived from the provided URL.

@styjdt
Copy link

styjdt commented Aug 19, 2021

Great! If you didn't propose this change, I would have done. This is really needed.
Can't wait for this to be included in the Docker image...

@styjdt
Copy link

styjdt commented Aug 19, 2021

@tcardonne Any chance that a new version of the Docker image including this change is published in the near future?

@jamesseiwert
Copy link

@tcardonne Any thoughts on this being bublished?

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.

3 participants