diff --git a/.github/workflows/demo.yml b/.github/workflows/demo.yml index c3bb009..51e1cd0 100644 --- a/.github/workflows/demo.yml +++ b/.github/workflows/demo.yml @@ -1,60 +1,47 @@ -on: [push, pull_request] +on: [ push, pull_request ] jobs: - single_key_demo: - strategy: - matrix: - os: [ubuntu-latest, macOS-latest, windows-latest] - runs-on: ${{ matrix.os }} - steps: - - uses: actions/checkout@v2 - - name: Setup key - uses: ./ - with: - ssh-private-key: | - ${{ secrets.DEMO_KEY }} - ${{ secrets.DEMO_KEY_2 }} - - multiple_keys_demo: + deployment_keys_demo: strategy: + fail-fast: false matrix: - os: [ubuntu-latest, macOS-latest] + os: [ ubuntu-latest, macOS-latest, windows-latest ] runs-on: ${{ matrix.os }} steps: - - uses: actions/checkout@v2 - - name: Setup key - uses: ./ - with: - ssh-private-key: ${{ secrets.DEMO_KEY }} + - uses: actions/checkout@v2 + - name: Setup key + uses: ./ + with: + ssh-private-key: | + ${{ secrets.MPDUDE_TEST_1_DEPLOY_KEY }} + ${{ secrets.MPDUDE_TEST_2_DEPLOY_KEY }} + - run: | + git clone https://github.com/mpdude/test-1.git test-1-http + git clone git@github.com:mpdude/test-1.git test-1-git + git clone ssh://git@github.com/mpdude/test-1.git test-1-git-ssh + git clone https://github.com/mpdude/test-2.git test-2-http + git clone git@github.com:mpdude/test-2.git test-2-git + git clone ssh://git@github.com/mpdude/test-2.git test-2-git-ssh + go get -v github.com/mpdude/test-2 docker_demo: - runs-on: ubuntu-latest + runs-on: ubuntu-latest container: image: ubuntu:latest steps: - - uses: actions/checkout@v2 - - run: apt update && apt install -y openssh-client - - name: Setup key - uses: ./ - with: - ssh-private-key: | - ${{ secrets.DEMO_KEY }} - ${{ secrets.DEMO_KEY_2 }} + - uses: actions/checkout@v2 + - run: apt update && apt install -y openssh-client git + - name: Setup key + uses: ./ + with: + ssh-private-key: | + ${{ secrets.MPDUDE_TEST_1_DEPLOY_KEY }} + ${{ secrets.MPDUDE_TEST_2_DEPLOY_KEY }} + - run: | + git clone https://github.com/mpdude/test-1.git test-1-http + git clone git@github.com:mpdude/test-1.git test-1-git + git clone ssh://git@github.com/mpdude/test-1.git test-1-git-ssh + git clone https://github.com/mpdude/test-2.git test-2-http + git clone git@github.com:mpdude/test-2.git test-2-git + git clone ssh://git@github.com/mpdude/test-2.git test-2-git-ssh - deployment_keys_demo: - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v2 - - name: Setup key - uses: ./ - with: - ssh-private-key: | - ${{ secrets.MPDUDE_TEST_1_DEPLOY_KEY }} - ${{ secrets.MPDUDE_TEST_2_DEPLOY_KEY }} - - run: | - git clone https://github.com/mpdude/test-1.git test-1-http - git clone git@github.com:mpdude/test-1.git test-1-git - git clone ssh://git@github.com/mpdude/test-1.git test-1-git-ssh - git clone https://github.com/mpdude/test-2.git test-2-http - git clone git@github.com:mpdude/test-2.git test-2-git - git clone ssh://git@github.com/mpdude/test-2.git test-2-git-ssh diff --git a/README.md b/README.md index 63fccf8..87c63a9 100644 --- a/README.md +++ b/README.md @@ -33,9 +33,9 @@ jobs: ... steps: - actions/checkout@v2 - # Make sure the @v0.5.0 matches the current version of the + # Make sure the @v0.5.3 matches the current version of the # action - - uses: webfactory/ssh-agent@v0.5.0 + - uses: webfactory/ssh-agent@v0.5.3 with: ssh-private-key: ${{ secrets.SSH_PRIVATE_KEY }} - ... other steps @@ -50,7 +50,7 @@ You can set up different keys as different secrets and pass them all to the acti ```yaml # ... contens as before - - uses: webfactory/ssh-agent@v0.5.0 + - uses: webfactory/ssh-agent@v0.5.3 with: ssh-private-key: | ${{ secrets.FIRST_KEY }} @@ -68,7 +68,7 @@ When using **Github deploy keys**, GitHub servers will accept the _first_ known To support picking the right key in this use case, this action scans _key comments_ and will set up extra Git and SSH configuration to make things work. -1. When creating the deploy key for a repository like `git@github.com:owner/repo.git` or `https://github.com/owner/repo`, put that URL into the key comment. +1. When creating the deploy key for a repository like `git@github.com:owner/repo.git` or `https://github.com/owner/repo`, put that URL into the key comment. (Hint: Try `ssh-keygen ... -C "git@github.com:owner/repo.git"`.) 2. After keys have been added to the agent, this action will scan the key comments. 3. For key comments containing such URLs, a Git config setting is written that uses [`url..insteadof`](https://git-scm.com/docs/git-config#Documentation/git-config.txt-urlltbasegtinsteadOf). It will redirect `git` requests to URLs starting with either `https://github.com/owner/repo` or `git@github.com:owner/repo` to a fake hostname/URL like `git@...some.hash...:owner/repo`. 4. An SSH configuration section is generated that applies to the fake hostname. It will map the SSH connection back to `github.com`, while at the same time pointing SSH to a file containing the appropriate key's public part. That will make SSH use the right key when connecting to GitHub.com. @@ -90,6 +90,46 @@ If the private key is not in the `PEM` format, you will see an `Error loading ke Use `ssh-keygen -p -f path/to/your/key -m pem` to convert your key file to `PEM`, but be sure to make a backup of the file first 😉. +## Additional Information for Particular Tools or Platforms + +If you know that your favorite tool or platform of choice requires extra tweaks or has some caveats when running with SSH, feel free to open a PR to amend this section here. + +### Container-based Workflows + +If you are using this action on container-based workflows, make sure the container has the necessary SSH binaries or package(s) installed. + +### Cargo's (Rust) Private Dependencies on Windows + +If you are using private repositories in your dependencies like this: + +``` +stuff = { git = "ssh://git@github.com/myorg/stuff.git", branch = "main" } +``` + +... you will need to change a configuration in the workflow for Windows machines in order to make cargo able to clone private repositories. + +There are 2 ways you can achieve this: + +1. Add this step once in your job **before** any cargo command: + +``` + - name: Update cargo config to use Git CLI + run: Set-Content -Path $env:USERPROFILE\.cargo\config.toml "[net]`ngit-fetch-with-cli = true" +``` + +This will configure Cargo to use the Git CLI as explained in the [Cargo's documentation](https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli). + +2. Alternatively you can set it to the environment variables for the entire workflow: + +``` +env: + CARGO_NET_GIT_FETCH_WITH_CLI: true +``` + +### Using Deploy Keys with Swift Package Manager + +`xcodebuild` by default uses Xcode's built-in Git tooling. If you want to use GitHub Deploy Keys as supported by this action, however, that version of Git will lack the necessary URL remapping. In this case, pass `-scmProvider system` to the `xcodebuild` command, as mentioned in [Apple's documentation](https://developer.apple.com/documentation/swift_packages/building_swift_packages_or_apps_that_use_them_in_continuous_integration_workflows#3680255). + ## What this Action *cannot* do for you The following items are not issues, but beyond what this Action is supposed to do. diff --git a/action.yml b/action.yml index e43c44f..16aaba6 100644 --- a/action.yml +++ b/action.yml @@ -10,6 +10,7 @@ runs: using: 'node12' main: 'dist/index.js' post: 'dist/cleanup.js' + post-if: 'always()' branding: icon: loader color: 'yellow' diff --git a/cleanup.js b/cleanup.js index f90cddd..34b2bbc 100644 --- a/cleanup.js +++ b/cleanup.js @@ -1,10 +1,11 @@ -const core = require('@actions/core') -const { execSync } = require('child_process') +const core = require('@actions/core'); +const { execFileSync } = require('child_process'); +const { sshAgent } = require('./paths.js'); try { // Kill the started SSH agent - console.log('Stopping SSH agent') - execSync('kill ${SSH_AGENT_PID}', { stdio: 'inherit' }) + console.log('Stopping SSH agent'); + execFileSync(sshAgent, ['-k'], { stdio: 'inherit' }); } catch (error) { console.log(error.message); console.log('Error stopping the SSH agent, proceeding anyway'); diff --git a/dist/cleanup.js b/dist/cleanup.js index c8081be..38a3d56 100644 --- a/dist/cleanup.js +++ b/dist/cleanup.js @@ -122,13 +122,14 @@ module.exports = require("child_process"); /***/ 175: /***/ (function(__unusedmodule, __unusedexports, __webpack_require__) { -const core = __webpack_require__(470) -const { execSync } = __webpack_require__(129) +const core = __webpack_require__(470); +const { execFileSync } = __webpack_require__(129); +const { sshAgent } = __webpack_require__(972); try { // Kill the started SSH agent - console.log('Stopping SSH agent') - execSync('kill ${SSH_AGENT_PID}', { stdio: 'inherit' }) + console.log('Stopping SSH agent'); + execFileSync(sshAgent, ['-k'], { stdio: 'inherit' }); } catch (error) { console.log(error.message); console.log('Error stopping the SSH agent, proceeding anyway'); @@ -480,6 +481,31 @@ module.exports = require("path"); module.exports = require("fs"); +/***/ }), + +/***/ 972: +/***/ (function(module, __unusedexports, __webpack_require__) { + +const os = __webpack_require__(87); + +module.exports = (process.env['OS'] != 'Windows_NT') ? { + + // Use getent() system call, since this is what ssh does; makes a difference in Docker-based + // Action runs, where $HOME is different from the pwent + home: os.userInfo().homedir, + sshAgent: 'ssh-agent', + sshAdd: 'ssh-add' + +} : { + + home: os.homedir(), + sshAgent: 'c://progra~1//git//usr//bin//ssh-agent.exe', + sshAdd: 'c://progra~1//git//usr//bin//ssh-add.exe' + +}; + + + /***/ }) /******/ }); \ No newline at end of file diff --git a/dist/index.js b/dist/index.js index ac64344..6dbf026 100644 --- a/dist/index.js +++ b/dist/index.js @@ -118,8 +118,8 @@ exports.issueCommand = issueCommand; const core = __webpack_require__(470); const child_process = __webpack_require__(129); const fs = __webpack_require__(747); -const os = __webpack_require__(87); const crypto = __webpack_require__(417); +const { home, sshAgent, sshAdd } = __webpack_require__(972); try { const privateKey = core.getInput('ssh-private-key'); @@ -130,80 +130,77 @@ try { return; } - var home; - - if (process.env['OS'] == 'Windows_NT') { - console.log('Preparing ssh-agent service on Windows'); - child_process.execSync('sc config ssh-agent start=demand', { stdio: 'inherit' }); - - home = os.homedir(); - } else { - // Use getent() system call, since this is what ssh does; makes a difference in Docker-based - // Action runs, where $HOME is different from the pwent - var { homedir: home } = os.userInfo(); - } - const homeSsh = home + '/.ssh'; console.log(`Adding GitHub.com keys to ${homeSsh}/known_hosts`); + fs.mkdirSync(homeSsh, { recursive: true }); fs.appendFileSync(`${homeSsh}/known_hosts`, '\ngithub.com ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ==\n'); fs.appendFileSync(`${homeSsh}/known_hosts`, '\ngithub.com ssh-dss AAAAB3NzaC1kc3MAAACBANGFW2P9xlGU3zWrymJgI/lKo//ZW2WfVtmbsUZJ5uyKArtlQOT2+WRhcg4979aFxgKdcsqAYW3/LS1T2km3jYW/vr4Uzn+dXWODVk5VlUiZ1HFOHf6s6ITcZvjvdbp6ZbpM+DuJT7Bw+h5Fx8Qt8I16oCZYmAPJRtu46o9C2zk1AAAAFQC4gdFGcSbp5Gr0Wd5Ay/jtcldMewAAAIATTgn4sY4Nem/FQE+XJlyUQptPWMem5fwOcWtSXiTKaaN0lkk2p2snz+EJvAGXGq9dTSWHyLJSM2W6ZdQDqWJ1k+cL8CARAqL+UMwF84CR0m3hj+wtVGD/J4G5kW2DBAf4/bqzP4469lT+dF2FRQ2L9JKXrCWcnhMtJUvua8dvnwAAAIB6C4nQfAA7x8oLta6tT+oCk2WQcydNsyugE8vLrHlogoWEicla6cWPk7oXSspbzUcfkjN3Qa6e74PhRkc7JdSdAlFzU3m7LMkXo1MHgkqNX8glxWNVqBSc0YRdbFdTkL0C6gtpklilhvuHQCdbgB3LBAikcRkDp+FCVkUgPC/7Rw==\n'); console.log("Starting ssh-agent"); + const authSock = core.getInput('ssh-auth-sock'); - let sshAgentOutput = '' - if (authSock && authSock.length > 0) { - sshAgentOutput = child_process.execFileSync('ssh-agent', ['-a', authSock]); - } else { - sshAgentOutput = child_process.execFileSync('ssh-agent') - } + const sshAgentArgs = (authSock && authSock.length > 0) ? ['-a', authSock] : []; // Extract auth socket path and agent pid and set them as job variables - const lines = sshAgentOutput.toString().split("\n") - for (const lineNumber in lines) { - const matches = /^(SSH_AUTH_SOCK|SSH_AGENT_PID)=(.*); export \1/.exec(lines[lineNumber]) + child_process.execFileSync(sshAgent, sshAgentArgs).toString().split("\n").forEach(function(line) { + const matches = /^(SSH_AUTH_SOCK|SSH_AGENT_PID)=(.*); export \1/.exec(line); + if (matches && matches.length > 0) { + // This will also set process.env accordingly, so changes take effect for this script core.exportVariable(matches[1], matches[2]) + console.log(`${matches[1]}=${matches[2]}`); } - } + }); + + console.log("Adding private key(s) to agent"); - console.log("Adding private key to agent"); privateKey.split(/(?=-----BEGIN)/).forEach(function(key) { - child_process.execSync('ssh-add -', { input: key.trim() + "\n" }); + child_process.execFileSync(sshAdd, ['-'], { input: key.trim() + "\n" }); }); - console.log("Keys added:"); - child_process.execSync('ssh-add -l', { stdio: 'inherit' }); + console.log("Key(s) added:"); - child_process.execFileSync('ssh-add', ['-L']).toString().split(/\r?\n/).forEach(function(key) { - let parts = key.match(/\bgithub.com[:/](.*)(?:\.git)?\b/); + child_process.execFileSync(sshAdd, ['-l'], { stdio: 'inherit' }); + + console.log('Configuring deployment key(s)'); + + child_process.execFileSync(sshAdd, ['-L']).toString().split(/\r?\n/).forEach(function(key) { + const parts = key.match(/\bgithub\.com[:/]([_.a-z0-9-]+\/[_.a-z0-9-]+)/i); + + if (!parts) { + console.log(`Comment for key '${key}' does not match GitHub URL pattern. Not treating it as a GitHub deploy key.`); - if (parts == null) { return; } - let ownerAndRepo = parts[1]; - let sha256 = crypto.createHash('sha256').update(key).digest('hex'); + const sha256 = crypto.createHash('sha256').update(key).digest('hex'); + const ownerAndRepo = parts[1].replace(/\.git$/, ''); - fs.writeFileSync(`${homeSsh}/${sha256}`, key + "\n", { mode: '600' }); + fs.writeFileSync(`${homeSsh}/key-${sha256}`, key + "\n", { mode: '600' }); - child_process.execSync(`git config --global --replace-all url."git@${sha256}:${ownerAndRepo}".insteadOf "https://github.com/${ownerAndRepo}"`); - child_process.execSync(`git config --global --add url."git@${sha256}:${ownerAndRepo}".insteadOf "git@github.com:${ownerAndRepo}"`); - child_process.execSync(`git config --global --add url."git@${sha256}:${ownerAndRepo}".insteadOf "ssh://git@github.com/${ownerAndRepo}"`); + child_process.execSync(`git config --global --replace-all url."git@key-${sha256}.github.com:${ownerAndRepo}".insteadOf "https://github.com/${ownerAndRepo}"`); + child_process.execSync(`git config --global --add url."git@key-${sha256}.github.com:${ownerAndRepo}".insteadOf "git@github.com:${ownerAndRepo}"`); + child_process.execSync(`git config --global --add url."git@key-${sha256}.github.com:${ownerAndRepo}".insteadOf "ssh://git@github.com/${ownerAndRepo}"`); - let sshConfig = `\nHost ${sha256}\n` + const sshConfig = `\nHost key-${sha256}.github.com\n` + ` HostName github.com\n` - + ` User git\n` - + ` IdentityFile ${homeSsh}/${sha256}\n` + + ` IdentityFile ${homeSsh}/key-${sha256}\n` + ` IdentitiesOnly yes\n`; fs.appendFileSync(`${homeSsh}/config`, sshConfig); - console.log(`Added deploy-key mapping: Use key "${key}" for GitHub repository ${ownerAndRepo}`); + console.log(`Added deploy-key mapping: Use identity '${homeSsh}/key-${sha256}' for GitHub repository ${ownerAndRepo}`); }); } catch (error) { + + if (error.code == 'ENOENT') { + console.log(`The '${error.path}' executable could not be found. Please make sure it is on your PATH and/or the necessary packages are installed.`); + console.log(`PATH is set to: ${process.env.PATH}`); + } + core.setFailed(error.message); } @@ -567,6 +564,31 @@ module.exports = require("path"); module.exports = require("fs"); +/***/ }), + +/***/ 972: +/***/ (function(module, __unusedexports, __webpack_require__) { + +const os = __webpack_require__(87); + +module.exports = (process.env['OS'] != 'Windows_NT') ? { + + // Use getent() system call, since this is what ssh does; makes a difference in Docker-based + // Action runs, where $HOME is different from the pwent + home: os.userInfo().homedir, + sshAgent: 'ssh-agent', + sshAdd: 'ssh-add' + +} : { + + home: os.homedir(), + sshAgent: 'c://progra~1//git//usr//bin//ssh-agent.exe', + sshAdd: 'c://progra~1//git//usr//bin//ssh-add.exe' + +}; + + + /***/ }) /******/ }); \ No newline at end of file diff --git a/index.js b/index.js index cf7b562..1a4835e 100644 --- a/index.js +++ b/index.js @@ -1,8 +1,8 @@ const core = require('@actions/core'); const child_process = require('child_process'); const fs = require('fs'); -const os = require('os'); const crypto = require('crypto'); +const { home, sshAgent, sshAdd } = require('./paths.js'); try { const privateKey = core.getInput('ssh-private-key'); @@ -13,79 +13,76 @@ try { return; } - var home; - - if (process.env['OS'] == 'Windows_NT') { - console.log('Preparing ssh-agent service on Windows'); - child_process.execSync('sc config ssh-agent start=demand', { stdio: 'inherit' }); - - home = os.homedir(); - } else { - // Use getent() system call, since this is what ssh does; makes a difference in Docker-based - // Action runs, where $HOME is different from the pwent - var { homedir: home } = os.userInfo(); - } - const homeSsh = home + '/.ssh'; console.log(`Adding GitHub.com keys to ${homeSsh}/known_hosts`); + fs.mkdirSync(homeSsh, { recursive: true }); fs.appendFileSync(`${homeSsh}/known_hosts`, '\ngithub.com ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ==\n'); fs.appendFileSync(`${homeSsh}/known_hosts`, '\ngithub.com ssh-dss AAAAB3NzaC1kc3MAAACBANGFW2P9xlGU3zWrymJgI/lKo//ZW2WfVtmbsUZJ5uyKArtlQOT2+WRhcg4979aFxgKdcsqAYW3/LS1T2km3jYW/vr4Uzn+dXWODVk5VlUiZ1HFOHf6s6ITcZvjvdbp6ZbpM+DuJT7Bw+h5Fx8Qt8I16oCZYmAPJRtu46o9C2zk1AAAAFQC4gdFGcSbp5Gr0Wd5Ay/jtcldMewAAAIATTgn4sY4Nem/FQE+XJlyUQptPWMem5fwOcWtSXiTKaaN0lkk2p2snz+EJvAGXGq9dTSWHyLJSM2W6ZdQDqWJ1k+cL8CARAqL+UMwF84CR0m3hj+wtVGD/J4G5kW2DBAf4/bqzP4469lT+dF2FRQ2L9JKXrCWcnhMtJUvua8dvnwAAAIB6C4nQfAA7x8oLta6tT+oCk2WQcydNsyugE8vLrHlogoWEicla6cWPk7oXSspbzUcfkjN3Qa6e74PhRkc7JdSdAlFzU3m7LMkXo1MHgkqNX8glxWNVqBSc0YRdbFdTkL0C6gtpklilhvuHQCdbgB3LBAikcRkDp+FCVkUgPC/7Rw==\n'); console.log("Starting ssh-agent"); + const authSock = core.getInput('ssh-auth-sock'); - let sshAgentOutput = '' - if (authSock && authSock.length > 0) { - sshAgentOutput = child_process.execFileSync('ssh-agent', ['-a', authSock]); - } else { - sshAgentOutput = child_process.execFileSync('ssh-agent') - } + const sshAgentArgs = (authSock && authSock.length > 0) ? ['-a', authSock] : []; // Extract auth socket path and agent pid and set them as job variables - const lines = sshAgentOutput.toString().split("\n") - for (const lineNumber in lines) { - const matches = /^(SSH_AUTH_SOCK|SSH_AGENT_PID)=(.*); export \1/.exec(lines[lineNumber]) + child_process.execFileSync(sshAgent, sshAgentArgs).toString().split("\n").forEach(function(line) { + const matches = /^(SSH_AUTH_SOCK|SSH_AGENT_PID)=(.*); export \1/.exec(line); + if (matches && matches.length > 0) { + // This will also set process.env accordingly, so changes take effect for this script core.exportVariable(matches[1], matches[2]) + console.log(`${matches[1]}=${matches[2]}`); } - } + }); + + console.log("Adding private key(s) to agent"); - console.log("Adding private key to agent"); privateKey.split(/(?=-----BEGIN)/).forEach(function(key) { - child_process.execSync('ssh-add -', { input: key.trim() + "\n" }); + child_process.execFileSync(sshAdd, ['-'], { input: key.trim() + "\n" }); }); - console.log("Keys added:"); - child_process.execSync('ssh-add -l', { stdio: 'inherit' }); + console.log("Key(s) added:"); + + child_process.execFileSync(sshAdd, ['-l'], { stdio: 'inherit' }); + + console.log('Configuring deployment key(s)'); - child_process.execFileSync('ssh-add', ['-L']).toString().split(/\r?\n/).forEach(function(key) { - let parts = key.match(/\bgithub.com[:/](.*)(?:\.git)?\b/); + child_process.execFileSync(sshAdd, ['-L']).toString().split(/\r?\n/).forEach(function(key) { + const parts = key.match(/\bgithub\.com[:/]([_.a-z0-9-]+\/[_.a-z0-9-]+)/i); + + if (!parts) { + console.log(`Comment for key '${key}' does not match GitHub URL pattern. Not treating it as a GitHub deploy key.`); - if (parts == null) { return; } - let ownerAndRepo = parts[1]; - let sha256 = crypto.createHash('sha256').update(key).digest('hex'); + const sha256 = crypto.createHash('sha256').update(key).digest('hex'); + const ownerAndRepo = parts[1].replace(/\.git$/, ''); - fs.writeFileSync(`${homeSsh}/${sha256}`, key + "\n", { mode: '600' }); + fs.writeFileSync(`${homeSsh}/key-${sha256}`, key + "\n", { mode: '600' }); - child_process.execSync(`git config --global --replace-all url."git@${sha256}:${ownerAndRepo}".insteadOf "https://github.com/${ownerAndRepo}"`); - child_process.execSync(`git config --global --add url."git@${sha256}:${ownerAndRepo}".insteadOf "git@github.com:${ownerAndRepo}"`); - child_process.execSync(`git config --global --add url."git@${sha256}:${ownerAndRepo}".insteadOf "ssh://git@github.com/${ownerAndRepo}"`); + child_process.execSync(`git config --global --replace-all url."git@key-${sha256}.github.com:${ownerAndRepo}".insteadOf "https://github.com/${ownerAndRepo}"`); + child_process.execSync(`git config --global --add url."git@key-${sha256}.github.com:${ownerAndRepo}".insteadOf "git@github.com:${ownerAndRepo}"`); + child_process.execSync(`git config --global --add url."git@key-${sha256}.github.com:${ownerAndRepo}".insteadOf "ssh://git@github.com/${ownerAndRepo}"`); - let sshConfig = `\nHost ${sha256}\n` + const sshConfig = `\nHost key-${sha256}.github.com\n` + ` HostName github.com\n` - + ` User git\n` - + ` IdentityFile ${homeSsh}/${sha256}\n` + + ` IdentityFile ${homeSsh}/key-${sha256}\n` + ` IdentitiesOnly yes\n`; fs.appendFileSync(`${homeSsh}/config`, sshConfig); - console.log(`Added deploy-key mapping: Use key "${key}" for GitHub repository ${ownerAndRepo}`); + console.log(`Added deploy-key mapping: Use identity '${homeSsh}/key-${sha256}' for GitHub repository ${ownerAndRepo}`); }); } catch (error) { + + if (error.code == 'ENOENT') { + console.log(`The '${error.path}' executable could not be found. Please make sure it is on your PATH and/or the necessary packages are installed.`); + console.log(`PATH is set to: ${process.env.PATH}`); + } + core.setFailed(error.message); } diff --git a/paths.js b/paths.js new file mode 100644 index 0000000..8ee7afd --- /dev/null +++ b/paths.js @@ -0,0 +1,18 @@ +const os = require('os'); + +module.exports = (process.env['OS'] != 'Windows_NT') ? { + + // Use getent() system call, since this is what ssh does; makes a difference in Docker-based + // Action runs, where $HOME is different from the pwent + home: os.userInfo().homedir, + sshAgent: 'ssh-agent', + sshAdd: 'ssh-add' + +} : { + + home: os.homedir(), + sshAgent: 'c://progra~1//git//usr//bin//ssh-agent.exe', + sshAdd: 'c://progra~1//git//usr//bin//ssh-add.exe' + +}; +