.github/workflows/ftp.yml

@@ -2,7 +2,7 @@ name: FTP Test
 
 on:
   push:
-    branches: [master]
+    branches: [ master ]
 
 jobs:
   deploy:
@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: 🚚 Get latest code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v2.3.2
 
       - name: 📂 Sync files
         uses: ./
@@ -18,3 +18,4 @@ jobs:
           server: ftp.samkirkland.com
           username: test@samkirkland.com
           password: ${{ secrets.ftp_password }}
+

.github/workflows/ftps.yml

@@ -2,7 +2,7 @@ name: FTPS Test
 
 on:
   push:
-    branches: [master]
+    branches: [ master ]
 
 jobs:
   deploy:
@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: 🚚 Get latest code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v2.3.2
   
       - name: 📂 Sync files
         uses: ./
@@ -19,3 +19,4 @@ jobs:
           username: test@samkirkland.com
           password: ${{ secrets.ftp_password }}
           protocol: ftps
+          security: strict

README.md

@@ -20,10 +20,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: 🚚 Get latest code
-      uses: actions/checkout@v4
+      uses: actions/checkout@v2.3.2
     
     - name: 📂 Sync files
-      uses: SamKirkland/FTP-Deploy-Action@v4.3.5
+      uses: SamKirkland/FTP-Deploy-Action@4.0.0
       with:
         server: ftp.samkirkland.com
         username: myFtpUserName
@@ -33,7 +33,7 @@ jobs:
 ---
 
 ### Requirements
-- You must have ftp access to your server. If your host allows or requires ssh please use my [web-deploy](https://github.com/SamKirkland/web-deploy) action
+- You must have ftp access to your server. If your host requires ssh please use my web-deploy action (coming soon)
 - Some web hosts change the default port (21), check with your host for your port number
 
 ---
@@ -56,21 +56,20 @@ To add a `secret` go to the `Settings` tab in your project then select `Secrets`
 I strongly recommend you store your `password` as a secret.
 
 | Key Name                | Required | Example                    | Default Value                                           | Description                                                                                                                                                        |
-|-------------------------|----------|-------------------------------|-------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+|-------------------------|----------|----------------------------|---------------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------|
 | `server`                | Yes      | `ftp.samkirkland.com`      |                                                         | Deployment destination server                                                                                                                                      |
 | `username`              | Yes      | `username@samkirkland.com` |                                                         | FTP user name                                                                                                                                                      |
 | `password`              | Yes      | `CrazyUniquePassword&%123` |                                                         | FTP password, be sure to escape quotes and spaces                                                                                                                  |
 | `port`                  | No       | `990`                      | `21`                                                    | Server port to connect to (read your web hosts docs)                                                                                                               |
 | `protocol`              | No       | `ftps`                     | `ftp`                                                   | `ftp`: provides no encryption, `ftps`: full encryption newest standard (aka "explicit" ftps), `ftps-legacy`: full encryption legacy standard (aka "implicit" ftps) |
-| `local-dir`             | No       | `./myFolderToPublish/`        | `./`                          | Folder to upload from, must end with trailing slash `/`                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                              |
+| `local-dir`             | No       | `./myFolderToPublish/`     | `./`                                                    | Path to upload to on the server, must end with trailing slash `/`                                                                                                  |
-| `server-dir`            | No       | `public_html/www/`            | `./`                          | Folder to upload to (on the server), must end with trailing slash `/`                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                |
+| `server-dir`            | No       | `public_html/www/`         | `./`                                                    | Folder to upload from, must end with trailing slash `/`                                                                                                            |
 | `state-name`            | No       | `folder/.sync-state.json`  | `.ftp-deploy-sync-state.json`                           | Path and name of the state file - this file is used to track which files have been deployed                                                                        |
 | `dry-run`               | No       | `true`                     | `false`                                                 | Prints which modifications will be made with current config options, but doesn't actually make any changes                                                         |
 | `dangerous-clean-slate` | No       | `true`                     | `false`                                                 | Deletes ALL contents of server-dir, even items in excluded with 'exclude' argument                                                                                 |
-| `exclude`               | No       | [See Example](#exclude-files) | [See Example](#exclude-files) | An array of glob patterns, these files will not be included in the publish/delete process. [List MUST be in this format](#exclude-files). You can use [a glob tester](https://www.digitalocean.com/community/tools/glob?comments=true&glob=%2A%2A%2F.git%2A%2F%2A%2A&matches=false&tests=test%2Fsam&tests=.git%2F%0D&tests=.github%2F%0D&tests=.git%2Ftest%0D&tests=.gitattributes%0D&tests=.gitignore%0D&tests=.git%2Fconfig%0D&tests=.git%2Ftest%2Ftest&tests=.github%2Fworkflows%2Fmain.yml&tests=node_modules%2Ffolder%2F%0D&tests=node_modules%2Fotherfolder%2F%0D&tests=subfolder%2Fnode_modules%2F) to test your pattern(s). |
+| `exclude`               | No       |                            | `[.git*, .git*/**, node_modules/**, node_modules/**/*]` | An array of glob patterns, these files will not be included in the publish/delete process                                                                          |
 | `log-level`             | No       | `minimal`                  | `standard`                                              | `minimal`: only important info, `standard`: important info and basic file changes, `verbose`: print everything the script is doing                                 |
 | `security`              | No       | `strict`                   | `loose`                                                 | `strict`: Reject any connection which is not authorized with the list of supplied CAs. `loose`: Allow connection even when the domain is not certificate           |
-| `timeout`               | No       | `60000`                       | `30000`                       | Timeout in milliseconds for FTP operations                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                           |
 
 
 # Common Examples
@@ -86,12 +85,12 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: 🚚 Get latest code
-      uses: actions/checkout@v4
+      uses: actions/checkout@v2.3.2
 
-    - name: Use Node.js 16
+    - name: Use Node.js 12
-      uses: actions/setup-node@v2
+      uses: actions/setup-node@v2-beta
       with:
-        node-version: '16'
+        node-version: '12'
       
     - name: 🔨 Build Project
       run: |
@@ -99,7 +98,7 @@ jobs:
         npm run build
     
     - name: 📂 Sync files
-      uses: SamKirkland/FTP-Deploy-Action@v4.3.5
+      uses: SamKirkland/FTP-Deploy-Action@4.0.0
       with:
         server: ftp.samkirkland.com
         username: myFtpUserName
@@ -116,10 +115,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: 🚚 Get latest code
-      uses: actions/checkout@v4
+      uses: actions/checkout@v2.3.2
 
     - name: 📂 Sync files
-      uses: SamKirkland/FTP-Deploy-Action@v4.3.5
+      uses: SamKirkland/FTP-Deploy-Action@4.0.0
       with:
         server: ftp.samkirkland.com
         username: myFtpUserName
@@ -128,7 +127,7 @@ jobs:
         port: 1234 # todo replace with your web hosts ftps port
 ```
 
-#### Log only dry run: Use this option for testing
+#### Log only dry run: Use this mode for testing
 Ouputs a list of files that will be created/modified to sync your source without making any actual changes
 ```yml
 on: push
@@ -139,10 +138,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: 🚚 Get latest code
-      uses: actions/checkout@v4
+      uses: actions/checkout@v2.3.2
 
     - name: 📂 Sync files
-      uses: SamKirkland/FTP-Deploy-Action@v4.3.5
+      uses: SamKirkland/FTP-Deploy-Action@4.0.0
       with:
         server: ftp.samkirkland.com
         username: myFtpUserName
@@ -150,44 +149,6 @@ jobs:
         dry-run: true
 ```
 
-#### Exclude files
-Excludes files
-```yml
-on: push
-name: 🚀 Deploy website on push
-jobs:
-  web-deploy:
-    name: 🎉 Deploy
-    runs-on: ubuntu-latest
-    steps:
-    - name: 🚚 Get latest code
-      uses: actions/checkout@v4
-
-    - name: 📂 Sync files
-      uses: SamKirkland/FTP-Deploy-Action@v4.3.5
-      with:
-        server: ftp.samkirkland.com
-        username: myFtpUserName
-        password: ${{ secrets.password }}
-        exclude: |
-          **/.git*
-          **/.git*/**
-          **/node_modules/**
-          fileToExclude.txt
-```
-
-`exclude` has the following default value
-```yml
-exclude: |
-  **/.git*
-  **/.git*/**
-  **/node_modules/**
-```
-if you overwrite the default value you will probably want to respecify them
-
-
----
-
 _Want another example? Let me know by creating a [github issue](https://github.com/SamKirkland/FTP-Deploy-Action/issues/new)_
 
 ---
@@ -249,8 +210,7 @@ Git files are excluded by default! If you customize the `exclude` option make su
 You can use the `exclude` option to ignore specific files/folders from the publish. Keep in mind you will need to re-add the default exclude options if you want to keep them. For example the below option excludes all `.txt` files.
 
 ```yml
-exclude:
+exclude: '*.txt'
- - *.txt
 ```
 
 </details>

action.yml

@@ -1,52 +1,49 @@
-name: "FTP Deploy"
+name: 'FTP Deploy'
-description: "Automate deploying websites and more with this GitHub action via FTP and FTPS"
+description: 'Automate deploying websites and more with this GitHub action'
-author: "Sam Kirkland"
+author: 'Sam Kirkland'
 inputs:
     server:
       required: true
-    description: "ftp server"
+      description: 'ftp server'
     username:
       required: true
-    description: "ftp username"
+      description: 'ftp username'
     password:
       required: true
-    description: "ftp password"
+      description: 'ftp password'
     port:
       required: false
-    description: "Server port to connect to (read your web hosts docs)"
+      description: 'Server port to connect to (read your web hosts docs)'
     protocol:
       required: false
-    description: "protocol to deploy with - ftp, ftps, or ftps-legacy"
+      description: 'protocol to deploy with - ftp, ftps, or ftps-legacy'
     local-dir:
       required: false
-    description: "Folder to upload from, must end with trailing slash /"
+      description: 'Folder to upload from, must end with trailing slash /'
     server-dir:
       required: false
-    description: "Path to upload to on the server. Must end with trailing slash /"
+      description: 'Path to upload to on the server. Must end with trailing slash /'
     state-name:
       required: false
-    description: "Path and name of the state file - this file is used to track which files have been deployed"
+      description: 'Path and name of the state file - this file is used to track which files have been deployed'
     dry-run:
       required: false
-    description: "Prints which modifications will be made with current config options, but doesnt actually make any changes"
+      description: 'Prints which modifications will be made with current config options, but doesnt actually make any changes'
     dangerous-clean-slate:
       required: false
-    description: "Deletes ALL contents of server-dir, even items in excluded with exclude argument"
+      description: 'Deletes ALL contents of server-dir, even items in excluded with exclude argument'
     exclude:
       required: false
-    description: "An array of glob patterns, these files will not be included in the publish/delete process"
+      description: 'An array of glob patterns, these files will not be included in the publish/delete process'
     log-level:
       required: false
-    description: "How verbose should the information be - minimal, standard, or verbose"
+      description: 'How verbose should the information be - warn, info, debug'
     security:
       required: false
-    description: "strict or loose"
+      description: 'strict or loose'
-  timeout:
-    required: false
-    description: "Timeout in milliseconds for FTP operations"
 runs:
-  using: "node20"
+  using: 'node12'
-  main: "dist/index.js"
+  main: 'dist/index.js'
 branding:
-  icon: "upload-cloud"
+  icon: 'upload-cloud'
-  color: "blue"
+  color: 'orange'

jest.config.js

@@ -0,0 +1,3 @@
+module.exports = {
+    preset: "ts-jest"
+};

migration.md

@@ -1,40 +1,23 @@
-# How to migrate between versions
+# Migrating from v3 to v4
-
-## Migrating from v4.1.0 to v4.2.0
-
-`v4.2.0` parses the `exclude` option in a more standard way. Going forward the `exclude` option **must** be in the following format:
-
-```yml
-exclude: |
-  **/.git*
-  **/.git*/**
-  **/node_modules/**
-  fileToExclude.txt
-```
-
-## Migrating from v3 to v4
 
 Migrating from v3 to v4 should be fairly straightforward. Version 4 was designed with speed and ease of initial setup in mind. Going forward version 4 will be the only supported version.
 
-### Those who can't upgrade
+#### Those who can't upgrade
-
+Most features have been carried forward and improved upon. However, some features did not make the cut.
-Most features have been carried forward and improved upon. However, some features did not make the cut:
 - **`sftp` is no longer supported**. If you have `sftp` access you are using `ssh`, that means you have access to a much more modern and capable protocol. I plan on releasing a separate github action that will deploy over `sftp`/`ssh` using `rsync`. Until then you can continue using version 3.
 - The `include` argument has been removed. I didn't see much need for it in the initial release. If you need this feature please create a support ticket.
 
+---
+
 ### How to upgrade
 
-1. Remove `with: fetch-depth: 2`. It is no longer needed and removing it will _slightly_ speed up deployments.
+1) Remove `with: fetch-depth: 2`. It is no longer needed and removing it will _slightly_ speed up deployments.
-2. Change the version to `v4.X.X`, for example `SamKirkland/FTP-Deploy-Action@v4.3.5` (please check the [README](https://github.com/SamKirkland/FTP-Deploy-Action/blob/master/README.md) or the [releases page](https://github.com/SamKirkland/FTP-Deploy-Action/releases/latest) for the latest version).
+2) Change the version to `4.X.X`, for example `SamKirkland/FTP-Deploy-Action@4.0.0` (please check readme for latest version)
-3. If you have a `.git-ftp-include` file you should delete it. Version 4 tracks files differently and no longer needs this config file.
+3) If you have a `.git-ftp-include` file you should delete it. Version 4 tracks files differently and no longer needs this config file.
-4. If you have a `.git-ftp-ignore` file, you should transfer the options to the new `exclude` argument. **Note:** version 4 excludes any `.git*` and `node_modules/` files / folders by default.
+4) If you have a `.git-ftp-ignore` file, you should transfer the options to the new `exclude` argument. _Note: Version 4 excludes any `.git*` and `node_modules/` files/folders by default_
-5. Update your arguments to reflect the following changes:
+5) Update your arguments to reflect the following changes
-    - `ftp-server` was split into 4 arguments:
+    * `ftp-server` was split into 4 arguments. `server`, `port`, `protocol`, and `server-dir`. Transfer your config to these options as needed.
-        - `server`
+    * `ftp-username` was renamed to `username`
-        - `port`
+    * `ftp-password` was renamed to `password`
-        - `protocol`
+    * `local-dir` and `server-dir` now **must** end with `/`
-        - `server-dir`
+    * `git-ftp-args` and `known-hosts` arguments were removed
-    - `ftp-username` was renamed to `username`.
-    - `ftp-password` was renamed to `password`.
-    - `local-dir` and `server-dir` now **must** end with `/`.
-    - `git-ftp-args` and `known-hosts` arguments were removed.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ftp-deploy-action",
-  "version": "4.3.5",
+  "version": "4.0.0",
   "private": true,
   "description": "Automate deploying websites and more with this GitHub action",
   "main": "dist/index.js",
@@ -14,31 +14,27 @@
     "url": "git+https://github.com/SamKirkland/FTP-Deploy-Action.git"
   },
   "keywords": [
-    "ftp",
     "website deploy",
     "continuous integration",
+    "ftp",
     "ftps"
   ],
   "author": "Sam Kirkland",
   "license": "MIT",
   "dependencies": {
-    "@actions/core": "^1.9.1",
+    "@actions/core": "^1.2.6",
-    "@samkirkland/ftp-deploy": "^1.2.4",
+    "@samkirkland/ftp-deploy": "^1.0.3",
-    "@types/jest": "^29.4.1",
+    "@types/jest": "^26.0.15",
-    "jest": "^29.5.0",
+    "jest": "^26.6.3",
-    "ts-jest": "^29.0.5",
+    "ts-jest": "^26.4.4",
-    "ts-node-dev": "^2.0.0"
+    "ts-node-dev": "^1.0.0-pre.62"
   },
   "devDependencies": {
-    "@types/node": "^20.11.24",
+    "@types/node": "^14.0.27",
-    "@typescript-eslint/eslint-plugin": "^5.33.1",
+    "@typescript-eslint/parser": "^4.0.1",
-    "@typescript-eslint/parser": "^5.33.1",
+    "@vercel/ncc": "^0.24.0",
-    "@vercel/ncc": "^0.34.0",
+    "eslint": "^7.8.1",
-    "eslint": "^8.22.0",
+    "js-yaml": "^3.14.0",
-    "eslint-plugin-jest": "^26.8.7",
+    "typescript": "^4.0.2"
-    "typescript": "^4.7.4"
-  },
-  "jest": {
-    "preset": "ts-jest"
   }
 }

src/main.test.ts

@@ -1,4 +1,4 @@
-import { optionalBoolean, optionalInt, optionalLogLevel, optionalProtocol, optionalSecurity, optionalString } from "./parse";
+import { optionalBoolean, optionalInt, optionalLogLevel, optionalProtocol, optionalSecurity, optionalString, optionalStringArray } from "./parse";
 
 describe("boolean", () => {
     test("false", () => {
@@ -103,3 +103,33 @@ describe("security", () => {
         expect(optionalSecurity("test", "strict")).toBe("strict");
     });
 });
+
+describe("array", () => {
+    test("empty", () => {
+        expect(optionalStringArray("test", "")).toEqual(undefined);
+    });
+
+    test("empty array", () => {
+        expect(optionalStringArray("test", "[]")).toEqual([]);
+    });
+
+    test(`["test.txt"]`, () => {
+        expect(optionalStringArray("test", "[test.txt]")).toEqual(["test.txt"]);
+    });
+
+    test(`[ "test.txt" ]`, () => {
+        expect(optionalStringArray("test", "[ test.txt ]")).toEqual(["test.txt"]);
+    });
+
+    test(`["test.txt", "folder/**/*"]`, () => {
+        expect(optionalStringArray("test", "[test.txt, folder/**/*]")).toEqual(["test.txt", "folder/**/*"]);
+    });
+
+    test(`["test.txt", "folder/**/*", "*other"]`, () => {
+        expect(optionalStringArray("test", `test.txt\n  - folder/**/*\n  - *other`)).toEqual(["test.txt", "folder/**/*", "*other"]);
+    });
+
+    test(`["test.txt", "folder/**/*", "*other"]`, () => {
+        expect(optionalStringArray("test", `\n  - test.txt\n  - folder/**/*\n  - *other`)).toEqual(["test.txt", "folder/**/*", "*other"]);
+    });
+});

src/main.ts

@@ -16,15 +16,14 @@ async function runDeployment() {
       "state-name": optionalString(core.getInput("state-name")),
       "dry-run": optionalBoolean("dry-run", core.getInput("dry-run")),
       "dangerous-clean-slate": optionalBoolean("dangerous-clean-slate", core.getInput("dangerous-clean-slate")),
-      "exclude": optionalStringArray("exclude", core.getMultilineInput("exclude")),
+      "exclude": optionalStringArray("exclude", core.getInput("exclude")),
       "log-level": optionalLogLevel("log-level", core.getInput("log-level")),
-      "security": optionalSecurity("security", core.getInput("security")),
+      "security": optionalSecurity("security", core.getInput("security"))
-      "timeout": optionalInt("timeout", core.getInput("timeout"))
     };
 
     await deploy(args);
   }
-  catch (error: any) {
+  catch (error) {
     core.setFailed(error);
   }
 }

src/parse.ts

@@ -90,14 +90,24 @@ export function optionalInt(argumentName: string, rawValue: string): number | un
     throw new Error(`${argumentName}: invalid parameter - you provided "${rawValue}". Try a whole number (no decimals) instead like 1234`);
 }
 
-export function optionalStringArray(argumentName: string, rawValue: string[]): string[] | undefined {
+export function optionalStringArray(argumentName: string, rawValue: string): string[] | undefined {
     if (rawValue.length === 0) {
         return undefined;
     }
 
-    if (typeof rawValue === "string") {
+    const valueTrim = rawValue.trim();
-        throw new Error(`${argumentName}: invalid parameter - you provided "${rawValue}". This option expects an list in the EXACT format described in the readme`);
+
+    if (valueTrim.startsWith("[")) {
+        // remove [ and ] - then convert to array
+        return rawValue.replace(/[\[\]]/g, "").trim().split(", ").filter(str => str !== "");
     }
 
-    return rawValue;
+    // split value by space and comma
+    const valueAsArrayDouble = rawValue.split(" - ").map(str => str.trim()).filter(str => str !== "");
+
+    if (valueAsArrayDouble.length) {
+        return valueAsArrayDouble;
+    }
+
+    throw new Error(`${argumentName}: invalid parameter - you provided "${rawValue}". This option excepts an array in the format [val1, val2] or val1\/n - val2`);
 }

tsconfig.json

@@ -1,6 +1,5 @@
 {
   "compilerOptions": {
-    "moduleResolution": "Node",
     "target": "ES2019", /* Specify ECMAScript target version: 'ES3' (default), 'ES5', 'ES2015', 'ES2016', 'ES2017', 'ES2018', 'ES2019' or 'ESNEXT'. */
     "module": "commonjs", /* Specify module code generation: 'none', 'commonjs', 'amd', 'system', 'umd', 'es2015', or 'ESNext'. */
     "outDir": "./dist", /* Redirect output structure to the directory. */