Skip to content

enforce baseUrl to api.github.com if action used on GHES#192

Merged
crazy-max merged 3 commits intodocker:masterfrom
crazy-max:support-ghes
Jan 28, 2023
Merged

enforce baseUrl to api.github.com if action used on GHES#192
crazy-max merged 3 commits intodocker:masterfrom
crazy-max:support-ghes

Conversation

@crazy-max
Copy link
Copy Markdown
Member

follow-up #191 (comment)

We need to enforce the baseUrl in the Octokit client for GHES users otherwise the it will talk to the GHES API and not api.github.com

Also adds extra commit with a note in the README for GHES users and handle the github_token input correctly for them.

Thanks for the feedback @rmg!

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max crazy-max requested a review from tonistiigi January 28, 2023 01:53
Copy link
Copy Markdown
Member

@tonistiigi tonistiigi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rmg lgty?

Copy link
Copy Markdown

@rmg rmg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think that should do it!

@crazy-max
Copy link
Copy Markdown
Member Author

Thanks again @rmg!

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