ssh-config 4.1.5 → 4.1.6

package/Readme.md

@@ -108,12 +108,14 @@ config.find({ Host: 'example1' })
 config.find(line => line.param == 'Host' && line.value == 'example1')
 ```
 
-### `.remove` sections by Host or other criteria
+### `.remove` sections by Host / Match or function
 
 To remove sections, we can pass the section to `.remove(opts)`.
 
 ```js
 config.remove({ Host: 'example1' })
+// or the ES2015 Array.prototype.find
+config.remove(line => line.param == 'Host' && line.value == 'example1')
 ```
 
 ### `.append` sections

package/index.js

@@ -79,7 +79,7 @@ class SSHConfig extends Array {
   }
 
   /**
-   * find section by Host or Match
+   * find section by Host / Match or function
    */
   find(opts = {}) {
     if (typeof opts === 'function') return super.find(opts)
@@ -92,17 +92,22 @@ class SSHConfig extends Array {
   }
 
   /**
-   * Remove section
+   * Remove section by Host / Match or function
    */
   remove(opts = {}) {
-    if (!(opts && ('Host' in opts || 'Match' in opts))) {
-      throw new Error('Can only remove by Host or Match')
-    }
+    let index;
+
+    if (typeof opts === 'function') {
+      index = super.findIndex(opts);
+
+    } else if (!(opts && ('Host' in opts || 'Match' in opts))) {
+      throw new Error('Can only remove by Host or Match');
 
-    const index = typeof opts === 'function'
-      ? super.findIndex(opts)
-      : super.findIndex(line => compare(line, opts))
+    } else {
+      index = super.findIndex(line => compare(line, opts));
 
+    } 
+    
     if (index >= 0) return this.splice(index, 1)
   }
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ssh-config",
   "description": "SSH config parser and stringifier",
-  "version": "4.1.5",
+  "version": "4.1.6",
   "author": "Chen Yangjian (https://www.cyj.me)",
   "repository": {
     "type": "git",