ssh-config 4.1.0 → 4.1.3

package/History.md

@@ -1,3 +1,31 @@
+4.1.3 / 2022-03-11
+==================
+
+## What's Changed
+* fix: IdentityAgent should be quoted if necessary by @cyjake in https://github.com/cyjake/ssh-config/pull/52
+
+
+**Full Changelog**: https://github.com/cyjake/ssh-config/compare/v4.1.2...v4.1.3
+
+4.1.2 / 2022-01-20
+==================
+
+## What's Changed
+* docs: types field in package.json by @cyjake in https://github.com/cyjake/ssh-config/pull/50
+
+
+**Full Changelog**: https://github.com/cyjake/ssh-config/compare/v4.1.1...v4.1.2
+
+4.1.1 / 2021-10-21
+==================
+
+## What's Changed
+* docs: `.prepend` and type definitions by @cyjake in https://github.com/cyjake/ssh-config/pull/47
+* fix: improper parsing of ProxyCommand with quotation marks by @tanhakabir in https://github.com/cyjake/ssh-config/pull/48
+
+
+**Full Changelog**: https://github.com/cyjake/ssh-config/compare/v4.1.0...v4.1.1
+
 4.1.0 / 2021-10-20
 ==================
 

package/Readme.md

@@ -154,6 +154,62 @@ SSHConfig.stringify(config)
 //   User dinosaur
 ```
 
+### `.prepend` sections
+
+But appending options to the end of the config isn't very effective if your config is organizated per the recommendations of ssh_config(5) that the generic options are at at the end of the config, such as:
+
+```
+Host ness
+  HostName lochness.com
+  User dinosaur
+
+IdentityFile ~/.ssh/id_rsa
+```
+
+The config could get messy if you put new options after the line of `IdentityFile`. To work around this issue, it is recommended that `.prepend` should be used instead. For the example above, we can prepend new options at the beginning of the config:
+
+```js
+config.prepend({
+  Host: 'tahoe',
+  HostName 'tahoe.com',
+})
+```
+
+The result would be:
+
+```
+Host tahoe
+  HostName tahoe.com
+
+Host ness
+  HostName lochness.com
+  User dinosaur
+
+IdentityFile ~/.ssh/id_rsa
+```
+
+If there are generic options at the beginning of the config, and you'd like the prepended section put before the first existing section, please turn on the second argument of `.prepend`:
+
+```js
+config.prepend({
+  Host: 'tahoe',
+  HostName 'tahoe.com',
+}, true)
+```
+
+The result would be like:
+
+```
+IdentityFile ~/.ssh/id_rsa
+
+Host tahoe
+  HostName tahoe.com
+
+Host ness
+  HostName lochness.com
+  User dinosaur
+```
+
 ## References
 
 - [ssh_config(5)][ssh_config]

package/index.js

@@ -5,17 +5,13 @@ const glob = require('./src/glob')
 const RE_SPACE = /\s/
 const RE_LINE_BREAK = /\r|\n/
 const RE_SECTION_DIRECTIVE = /^(Host|Match)$/i
-const RE_MULTI_VALUE_DIRECTIVE = /^(GlobalKnownHostsFile|Host|IPQoS|SendEnv|UserKnownHostsFile)$/i
-const RE_QUOTE_DIRECTIVE = /^(?:CertificateFile|IdentityFile|User)$/i
+const RE_MULTI_VALUE_DIRECTIVE = /^(GlobalKnownHostsFile|Host|IPQoS|SendEnv|UserKnownHostsFile|ProxyCommand)$/i
+const RE_QUOTE_DIRECTIVE = /^(?:CertificateFile|IdentityFile|IdentityAgent|User)$/i
 const RE_SINGLE_LINE_DIRECTIVE = /^(Include|IdentityFile)$/i
 
 const DIRECTIVE = 1
 const COMMENT = 2
 
-function compare(line, opts) {
-  return opts.hasOwnProperty(line.param) && opts[line.param] === line.value
-}
-
 const MULTIPLE_VALUE_PROPS = [
   'IdentityFile',
   'LocalForward',
@@ -24,6 +20,24 @@ const MULTIPLE_VALUE_PROPS = [
   'CertificateFile'
 ]
 
+function compare(line, opts) {
+  return opts.hasOwnProperty(line.param) && opts[line.param] === line.value
+}
+
+function getIndent(config) {
+  for (const line of config) {
+    if (RE_SECTION_DIRECTIVE.test(line.param)) {
+      for (const subline of line.config) {
+        if (subline.before) {
+          return subline.before
+        }
+      }
+    }
+  }
+
+  return '  '
+}
+
 class SSHConfig extends Array {
   /**
    * Query ssh config by host.
@@ -106,20 +120,7 @@ class SSHConfig extends Array {
    * @param {Object} opts
    */
   append(opts) {
-    let indent = '  '
-
-    outer:
-    for (const line of this) {
-      if (RE_SECTION_DIRECTIVE.test(line.param)) {
-        for (const subline of line.config) {
-          if (subline.before) {
-            indent = subline.before
-            break outer
-          }
-        }
-      }
-    }
-
+    const indent = getIndent(this)
     const lastEntry = this.length > 0 ? this[this.length - 1] : null
     let config = lastEntry && lastEntry.config || this
     let configWas = this
@@ -158,21 +159,8 @@ class SSHConfig extends Array {
    * Prepend new section to existing ssh config.
    * @param {Object} opts
    */
-   prepend(opts, beforeFirstSection = false) {
-    let indent = '  '
-
-    outer:
-    for (const line of this) {
-      if (RE_SECTION_DIRECTIVE.test(line.param)) {
-        for (const subline of line.config) {
-          if (subline.before) {
-            indent = subline.before
-            break outer
-          }
-        }
-      }
-    }
-
+  prepend(opts, beforeFirstSection = false) {
+    const indent = getIndent(this)
     let config = this
     let i = 0
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ssh-config",
   "description": "SSH config parser and stringifier",
-  "version": "4.1.0",
+  "version": "4.1.3",
   "author": "Chen Yangjian (https://www.cyj.me)",
   "repository": {
     "type": "git",
@@ -9,7 +9,8 @@
   },
   "files": [
     "index.js",
-    "src"
+    "src",
+    "types"
   ],
   "devDependencies": {
     "eslint": "^7.17.0",
@@ -25,5 +26,6 @@
   "engine": {
     "node": ">= 10.0.0"
   },
+  "types": "types/index.d.ts",
   "license": "MIT"
 }

package/types/index.d.ts

@@ -0,0 +1,41 @@
+declare enum ELine {
+  DIRECTIVE = 1,
+  COMMENT = 2,
+}
+
+interface Directive {
+  type: ELine.DIRECTIVE;
+  before: string;
+  after: string;
+  param: string;
+  separator: ' ' | '=';
+  value: string;
+}
+
+interface Section extends Directive {
+  config: SSHConfig;
+}
+
+interface Comment {
+  type: ELine.COMMENT;
+  content: string;
+}
+
+type Line = Directive | Comment;
+
+export default class SSHConfig extends Array {
+  static parse(text: string): SSHConfig;
+  static stringify(config: SSHConfig): string;
+
+  toString(): string;
+
+  compute(host: string): Record<string, string>;
+
+  find(predicate: (value: any, index: number, obj: any[]) => any);
+  find(options: Record<string, string>): Line | Section;
+
+  remove(options: Record<string, string>): Line | Section;
+
+  append(options: Record<string, string>): SSHConfig;
+  prepend(options: Record<string, string>): SSHConfig;
+}

package/types/ssh-config-tests.ts

@@ -0,0 +1,10 @@
+import SSHConfig from './index';
+
+const config = SSHConfig.parse(`
+IdentityFile ~/.ssh/id_rsa
+
+Host ness
+  HostName lochness.com
+`);
+
+console.log(config.toString());