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

Allow GHES repository to support deploy key #680

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jiminj
Copy link

@jiminj jiminj commented Dec 15, 2021

This PR is a revision of #679, allowing to deploy on GHES repositories using deploy keys. - Found that simple error code checking does not handle the case when ssh-keyscan is failed. this PR includes the updated logic and corresponding test codes.

Tested with the GHES server with self-hosted runners, and github.com. You can test it by using uses: jiminj/[email protected]

@jiminj
Copy link
Author

jiminj commented Dec 15, 2021

BTW, I have another fix related to #579 (comment). Will make another PR once this is resolved.

@peaceiris peaceiris added this to the v3.9.0 milestone Dec 21, 2021
@jiminj jiminj mentioned this pull request Dec 22, 2021
3 tasks
@haroldo-bonette
Copy link

@peaceiris it would be nice to have this fix incorporated

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