diff --git a/lib/actionparameters.js b/lib/actionparameters.js index 8851e5931..7452f495b 100644 --- a/lib/actionparameters.js +++ b/lib/actionparameters.js @@ -32,7 +32,10 @@ class ActionParameters { this._images = core.getInput('images'); this._multiContainerConfigFile = core.getInput('configuration-file'); this._startupCommand = core.getInput('startup-command'); - this._commitMessage = github.context.eventName === 'push' ? github.context.payload.head_commit.message.split(/\r?\n/)[0] : ""; + /** + * Trimming the commit message because it is used as a param in uri of deployment api. And sometimes, it exceeds the max length of http URI. + */ + this._commitMessage = github.context.eventName === 'push' ? github.context.payload.head_commit.message.slice(0, 7000) : ""; this._endpoint = endpoint; } static getActionParams(endpoint) { diff --git a/node_modules/resolve/bin/resolve b/node_modules/resolve/bin/resolve new file mode 100755 index 000000000..5ee329af9 --- /dev/null +++ b/node_modules/resolve/bin/resolve @@ -0,0 +1,50 @@ +#!/usr/bin/env node + +'use strict'; + +var path = require('path'); +var fs = require('fs'); + +if ( + String(process.env.npm_lifecycle_script).slice(0, 8) !== 'resolve ' + && ( + !process.argv + || process.argv.length < 2 + || (process.argv[1] !== __filename && fs.statSync(process.argv[1]).ino !== fs.statSync(__filename).ino) + || (process.env._ && path.resolve(process.env._) !== __filename) + ) +) { + console.error('Error: `resolve` must be run directly as an executable'); + process.exit(1); +} + +var supportsPreserveSymlinkFlag = require('supports-preserve-symlinks-flag'); + +var preserveSymlinks = false; +for (var i = 2; i < process.argv.length; i += 1) { + if (process.argv[i].slice(0, 2) === '--') { + if (supportsPreserveSymlinkFlag && process.argv[i] === '--preserve-symlinks') { + preserveSymlinks = true; + } else if (process.argv[i].length > 2) { + console.error('Unknown argument ' + process.argv[i].replace(/[=].*$/, '')); + process.exit(2); + } + process.argv.splice(i, 1); + i -= 1; + if (process.argv[i] === '--') { break; } // eslint-disable-line no-restricted-syntax + } +} + +if (process.argv.length < 3) { + console.error('Error: `resolve` expects a specifier'); + process.exit(2); +} + +var resolve = require('../'); + +var result = resolve.sync(process.argv[2], { + basedir: process.cwd(), + preserveSymlinks: preserveSymlinks +}); + +console.log(result); diff --git a/src/actionparameters.ts b/src/actionparameters.ts index 4bd4a1c4c..fc2677182 100644 --- a/src/actionparameters.ts +++ b/src/actionparameters.ts @@ -43,15 +43,17 @@ export class ActionParameters { this._images = core.getInput('images'); this._multiContainerConfigFile = core.getInput('configuration-file'); this._startupCommand = core.getInput('startup-command'); - this._commitMessage = github.context.eventName === 'push'? github.context.payload.head_commit.message.split(/\r?\n/)[0] : ""; - this._endpoint = endpoint; + /** + * Trimming the commit message because it is used as a param in uri of deployment api. And sometimes, it exceeds the max length of http URI. + */ + this._commitMessage = github.context.eventName === 'push' ? github.context.payload.head_commit.message.slice(0, 7000) : ""; + this._endpoint = endpoint; } public static getActionParams(endpoint?: IAuthorizer) { if (!this.actionparams) { this.actionparams = new ActionParameters(!!endpoint ? endpoint : null); } - return this.actionparams; } public get appName() {