Error on deployment... worked well up until now #118

karfu-git · 2023-02-21T15:15:07Z

Hi! I've been using this package as part of my deployment process for some time now, today I got the following error when my pipeline ran...

/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                  
TypeError: Cannot read properties of undefined (reading 'split')
    at /home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:4450
    at Array.forEach (<anonymous>)
    at Object.229 (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:43[19](https://github.com/karfu-git/Laravel-Ingester/actions/runs/4233945900/jobs/7355630423#step:12:20))
    at __nccwpck_require__ (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:8812)
    at Object.976 (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:4805)
    at __nccwpck_require__ (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:8812)
    at /home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:9078
    at /home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:9799
    at Object.<anonymous> (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:98[20](https://github.com/karfu-git/Laravel-Ingester/actions/runs/4233945900/jobs/7355630423#step:12:21))
    at Module._compile (node:internal/modules/cjs/loader:1105:14)

The text was updated successfully, but these errors were encountered:

williamspencer · 2023-02-21T15:23:22Z

Hi! I've been using this package as part of my deployment process for some time now, today I got the following error when my pipeline ran...

/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2
                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                  
TypeError: Cannot read properties of undefined (reading 'split')
    at /home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:4450
    at Array.forEach (<anonymous>)
    at Object.229 (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:43[19](https://github.com/karfu-git/Laravel-Ingester/actions/runs/4233945900/jobs/7355630423#step:12:20))
    at __nccwpck_require__ (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:8812)
    at Object.976 (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:4805)
    at __nccwpck_require__ (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:8812)
    at /home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:9078
    at /home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:9799
    at Object.<anonymous> (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:98[20](https://github.com/karfu-git/Laravel-Ingester/actions/runs/4233945900/jobs/7355630423#step:12:21))
    at Module._compile (node:internal/modules/cjs/loader:1105:14)

can confirm I'm experiencing same issue as well.

A0dh · 2023-02-21T15:28:19Z

Same issue on my end

A0dh · 2023-02-21T15:30:13Z

Using the version easingthemes/ssh-deploy@v4.1.0 for now. Seems to be working fine.

karfu-git · 2023-02-21T15:37:34Z

Using the version easingthemes/ssh-deploy@v4.1.0 for now. Seems to be working fine.

Well caught @A0dh ..Up and running again

ivansivaklepsi · 2023-02-21T15:51:54Z

Same is happening for me, suddenly the deploy stopped working

easingthemes · 2023-02-21T16:06:55Z

I'll check now

…heck fix: #118 check undefined default values

## [4.1.5](v4.1.4...v4.1.5) (2023-02-21) ### Bug Fixes * [#118](#118) check undefined default values ([f0c02fb](f0c02fb))

easingthemes · 2023-02-21T16:14:09Z

Update merged, check with @ main or v4.1.5

easingthemes · 2023-02-21T16:15:21Z

Issue is auto closed with a merge, reopen if it still happens.

easingthemes · 2023-02-21T16:16:23Z

Can you please also add your setup / list of envs, so I can add this case in e2e tests

…heck Bugfix/#118 default value check

## [4.1.6](v4.1.5...v4.1.6) (2023-02-21) ### Bug Fixes * [#118](#118) check undefined default values rebuild ([54f8b6c](54f8b6c))

karfu-git · 2023-02-21T17:06:31Z

Can you please also add your setup / list of envs, so I can add this case in e2e tests

Sure, my set up is as follows...

SSH_PRIVATE_KEY: ${{ secrets.KEY }}
REMOTE_HOST: ${{ env.HOST }}
REMOTE_USER: ${{ secrets.USER }}
TARGET: ${{ env.TARGET }}

My remote host is an AWS EC2, if that helps at all, so just using the hostname for that particular parameter.

Thanks for looking at this so quickly!

# 1.0.0 (2023-03-20) ### Bug Fixes * [easingthemes#113](https://github.com/progmonster/ssh-deploy/issues/113) limit ssh script input ([5894f5e](5894f5e)) * [easingthemes#113](https://github.com/progmonster/ssh-deploy/issues/113) limit ssh script output ([59827af](59827af)) * [easingthemes#113](https://github.com/progmonster/ssh-deploy/issues/113) limit ssh script output - rebuild ([756a522](756a522)) * [easingthemes#118](https://github.com/progmonster/ssh-deploy/issues/118) check undefined default values ([f0c02fb](f0c02fb)) * [easingthemes#118](https://github.com/progmonster/ssh-deploy/issues/118) check undefined default values rebuild ([54f8b6c](54f8b6c)) * [easingthemes#120](https://github.com/progmonster/ssh-deploy/issues/120) check undefined data ([2fbb060](2fbb060)) * add assets to semantic-release git ([0867b12](0867b12)) * add branch instead of a version in a readme ([8218c8e](8218c8e)) * add githubWorkspace as default source root ([3ef66d0](3ef66d0)) * Add manual inputs for e2e test ([8f71595](8f71595)) * add NPM token ([2197bc6](2197bc6)) * add package json auto updates ([ce56d75](ce56d75)) * Add visible Rsync CMD for debugging ([91b6d28](91b6d28)) * change npm to public ([191c82d](191c82d)) * codeql build ([5e456a4](5e456a4)) * codeql yaml syntax update ([87ad671](87ad671)) * fix default source if empty ([9f191f4](9f191f4)) * fix default TARGET ([4d08634](4d08634)) * fix manual release action ([245b7a9](245b7a9)) * log buffer from rsync stdout ([8e19e0d](8e19e0d)) * Log rsync command ([85f4a96](85f4a96)) * minify dist ([87551c3](87551c3)) * move e2e tests to main branch ([0069fae](0069fae)) * move e2e tests typo branch name ([6d3554b](6d3554b)) * only=prod package json ([95f4dc8](95f4dc8)) * plugins versions ([2f40dde](2f40dde)) * readme update ([80a7f53](80a7f53)), closes [easingthemes#67](https://github.com/progmonster/ssh-deploy/issues/67) * rebuild ([2169399](2169399)) * rebuild and update readme ([98025d6](98025d6)) * remove change npm to public ([3c36af7](3c36af7)) * replace i with ci ([50d1f57](50d1f57)) * semantic-release/npm update config ([40096bf](40096bf)) * Update changelog ([765f5ff](765f5ff)) * update ncc ([20a0cae](20a0cae)) * update npm package name ([9c4e411](9c4e411)) * update plugins ([b2adc00](b2adc00)) * update plugins ([a50a999](a50a999)) * update sub dependencies ([2a32c8d](2a32c8d)) * update v3 e2e test ([450bf42](450bf42)) * use main branch for e2e testes ([21451fe](21451fe)) * yaml step name ([5c13615](5c13615)) * 修复 ARGS 为空时导致 rsync 执行错误问题。 ([caaa95b](caaa95b)) * Merge pull request easingthemes#83 from easingthemes/feature/easingthemes#82-update-node-version ([76660a8](76660a8)), closes [easingthemes#83](https://github.com/progmonster/ssh-deploy/issues/83) [easingthemes/feature/easingthemes#82](https://github.com/easingthemes/feature//issues/82) ### Features * add e2e tests ([40f855a](40f855a)) * Add multi source and multi target support ([73a65ec](73a65ec)) * add semantic-release-action ([ac3c9b5](ac3c9b5)) * add ssh port from VARs, fix package-lock ([808b002](808b002)) * Add SSH remote script support - before and after rsync ([2cd8a82](2cd8a82)) * Add SSH_CMD_ARGS support ([51a0063](51a0063)) ### Performance Improvements * update default rsync options ([2be7efb](2be7efb)) ### BREAKING CHANGES * Default rsync options updated to speed up default deployments. from `rltgoDzvO` to `-rlgoDzvc -i` * update to use nodeJS v16

# 1.0.0 (2024-04-19) ### Bug Fixes * [easingthemes#113](https://github.com/ximplez/ssh-deploy/issues/113) limit ssh script input ([5894f5e](5894f5e)) * [easingthemes#113](https://github.com/ximplez/ssh-deploy/issues/113) limit ssh script output ([59827af](59827af)) * [easingthemes#113](https://github.com/ximplez/ssh-deploy/issues/113) limit ssh script output - rebuild ([756a522](756a522)) * [easingthemes#118](https://github.com/ximplez/ssh-deploy/issues/118) check undefined default values ([f0c02fb](f0c02fb)) * [easingthemes#118](https://github.com/ximplez/ssh-deploy/issues/118) check undefined default values rebuild ([54f8b6c](54f8b6c)) * [easingthemes#120](https://github.com/ximplez/ssh-deploy/issues/120) check undefined data ([2fbb060](2fbb060)) * add assets to semantic-release git ([0867b12](0867b12)) * add branch instead of a version in a readme ([8218c8e](8218c8e)) * add compiled file ([627ac29](627ac29)) * add githubWorkspace as default source root ([3ef66d0](3ef66d0)) * Add info for Permission denied issue. ([845b578](845b578)) * Add manual inputs for e2e test ([8f71595](8f71595)) * add NPM token ([2197bc6](2197bc6)) * add package json auto updates ([ce56d75](ce56d75)) * add uuid for ssh scripts ([66f6e4b](66f6e4b)) * Add visible Rsync CMD for debugging ([91b6d28](91b6d28)) * added the missing declarations [easingthemes#177](https://github.com/ximplez/ssh-deploy/issues/177) ([bb271fe](bb271fe)) * change npm to public ([191c82d](191c82d)) * codeql build ([5e456a4](5e456a4)) * codeql yaml syntax update ([87ad671](87ad671)) * fix default source if empty ([9f191f4](9f191f4)) * fix default TARGET ([4d08634](4d08634)) * Fix default values used incorrectly. ([a1b383f](a1b383f)) * fix manual release action ([245b7a9](245b7a9)) * log buffer from rsync stdout ([8e19e0d](8e19e0d)) * Log rsync command ([85f4a96](85f4a96)) * minify dist ([87551c3](87551c3)) * move e2e tests to main branch ([0069fae](0069fae)) * move e2e tests typo branch name ([6d3554b](6d3554b)) * normalize line endings in SSH key for the underlying OS ([3f5d9aa](3f5d9aa)) * only=prod package json ([95f4dc8](95f4dc8)) * plugins versions ([2f40dde](2f40dde)) * readme update ([80a7f53](80a7f53)), closes [easingthemes#67](https://github.com/ximplez/ssh-deploy/issues/67) * rebuild ([2169399](2169399)) * rebuild and update readme ([98025d6](98025d6)) * remove change npm to public ([3c36af7](3c36af7)) * replace i with ci ([50d1f57](50d1f57)) * semantic-release/npm update config ([40096bf](40096bf)) * trigger automated release ([4d8bbf0](4d8bbf0)) * Update changelog ([765f5ff](765f5ff)) * update ncc ([20a0cae](20a0cae)) * update npm package name ([9c4e411](9c4e411)) * update plugins ([b2adc00](b2adc00)) * update plugins ([a50a999](a50a999)) * update sub dependencies ([2a32c8d](2a32c8d)) * update v3 e2e test ([450bf42](450bf42)) * use main branch for e2e testes ([21451fe](21451fe)) * yaml step name ([5c13615](5c13615)) * 修复 ARGS 为空时导致 rsync 执行错误问题。 ([caaa95b](caaa95b)) * Merge pull request easingthemes#173 from jeromelachaud/main ([ac1908e](ac1908e)), closes [easingthemes#173](https://github.com/ximplez/ssh-deploy/issues/173) * Merge pull request easingthemes#83 from easingthemes/feature/easingthemes#82-update-node-version ([76660a8](76660a8)), closes [easingthemes#83](https://github.com/ximplez/ssh-deploy/issues/83) [easingthemes/feature/easingthemes#82](https://github.com/easingthemes/feature//issues/82) ### Features * add e2e tests ([40f855a](40f855a)) * Add multi source and multi target support ([73a65ec](73a65ec)) * add semantic-release-action ([ac3c9b5](ac3c9b5)) * add ssh port from VARs, fix package-lock ([808b002](808b002)) * Add SSH remote script support - before and after rsync ([2cd8a82](2cd8a82)) * Add SSH_CMD_ARGS support ([51a0063](51a0063)) ### Performance Improvements * update default rsync options ([2be7efb](2be7efb)) ### BREAKING CHANGES * update to use nodeJS v20 * update to use nodeJS v20 * Default rsync options updated to speed up default deployments. from `rltgoDzvO` to `-rlgoDzvc -i` * update to use nodeJS v16

easingthemes closed this as completed in f0c02fb Feb 21, 2023

easingthemes added a commit that referenced this issue Feb 21, 2023

Merge pull request #119 from easingthemes/bugfix/#118-default-value-c…

e7b2312

…heck fix: #118 check undefined default values

github-actions bot pushed a commit that referenced this issue Feb 21, 2023

chore(release): 4.1.5 [skip ci]

d9d134f

## [4.1.5](v4.1.4...v4.1.5) (2023-02-21) ### Bug Fixes * [#118](#118) check undefined default values ([f0c02fb](f0c02fb))

easingthemes mentioned this issue Feb 21, 2023

Error reading inputs SSH privates. #120

Closed

easingthemes added a commit that referenced this issue Feb 21, 2023

fix: #118 check undefined default values rebuild

54f8b6c

easingthemes added a commit that referenced this issue Feb 21, 2023

Merge branch 'main' into bugfix/#118-default-value-check

509caf1

easingthemes added a commit that referenced this issue Feb 21, 2023

Merge pull request #121 from easingthemes/bugfix/#118-default-value-c…

530a369

…heck Bugfix/#118 default value check

github-actions bot pushed a commit that referenced this issue Feb 21, 2023

chore(release): 4.1.6 [skip ci]

51b2009

## [4.1.6](v4.1.5...v4.1.6) (2023-02-21) ### Bug Fixes * [#118](#118) check undefined default values rebuild ([54f8b6c](54f8b6c))

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Error on deployment... worked well up until now #118

Error on deployment... worked well up until now #118

karfu-git commented Feb 21, 2023

williamspencer commented Feb 21, 2023

A0dh commented Feb 21, 2023

A0dh commented Feb 21, 2023

karfu-git commented Feb 21, 2023 •

edited

ivansivaklepsi commented Feb 21, 2023

easingthemes commented Feb 21, 2023

easingthemes commented Feb 21, 2023 •

edited

easingthemes commented Feb 21, 2023

easingthemes commented Feb 21, 2023

karfu-git commented Feb 21, 2023

Error on deployment... worked well up until now #118

Error on deployment... worked well up until now #118

Comments

karfu-git commented Feb 21, 2023

williamspencer commented Feb 21, 2023

A0dh commented Feb 21, 2023

A0dh commented Feb 21, 2023

karfu-git commented Feb 21, 2023 • edited

ivansivaklepsi commented Feb 21, 2023

easingthemes commented Feb 21, 2023

easingthemes commented Feb 21, 2023 • edited

easingthemes commented Feb 21, 2023

easingthemes commented Feb 21, 2023

karfu-git commented Feb 21, 2023

karfu-git commented Feb 21, 2023 •

edited

easingthemes commented Feb 21, 2023 •

edited